Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

updated & relinted #70

Merged
merged 1 commit into from Dec 6, 2023
Merged

Conversation

fredbi
Copy link
Member

@fredbi fredbi commented Dec 4, 2023

  • updated go version, in line with other go-openapi packages
  • updated golangci-lint config
  • relinted (essentially test code)
  • removed code guards with build tags for pre19
  • updated github actions
  • updated README (badges)

@fredbi fredbi requested a review from casualjim December 4, 2023 21:51
@fredbi fredbi force-pushed the chore/update-linting branch 2 times, most recently from dba2685 to 5033126 Compare December 4, 2023 22:08
Copy link

codecov bot commented Dec 5, 2023

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (4de0676) 84.94% compared to head (4f6e589) 86.37%.

Files Patch % Lines
yaml.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #70      +/-   ##
==========================================
+ Coverage   84.94%   86.37%   +1.43%     
==========================================
  Files          13       12       -1     
  Lines        1587     1585       -2     
==========================================
+ Hits         1348     1369      +21     
+ Misses        207      183      -24     
- Partials       32       33       +1     
Flag Coverage Δ
oldstable 86.37% <60.00%> (?)
stable 86.37% <60.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@fredbi fredbi force-pushed the chore/update-linting branch 2 times, most recently from 67e2a01 to 793d6e2 Compare December 5, 2023 21:16
@fredbi
Copy link
Member Author

fredbi commented Dec 5, 2023

builds on top of #71, to separate the fix on windows that prevented to run a CI on all platforms

* updated go version, in line with other go-openapi packages
* updated golangci-lint config
* relinted (essentially test code)
* removed code guards with build tags for pre19
* updated github actions, added run on windows
* updated README (badges)

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@fredbi fredbi merged commit cd381d7 into go-openapi:master Dec 6, 2023
9 of 10 checks passed
@fredbi fredbi deleted the chore/update-linting branch December 6, 2023 13:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants