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

chore: relint & update #269

Merged
merged 1 commit into from Dec 8, 2023
Merged

Conversation

fredbi
Copy link
Member

@fredbi fredbi commented Dec 7, 2023

  • updated go version requirement to go1.19

  • updated dependencies, excepted:

    • yaml (upgrade to v3 deferred to another PR)
    • otel (upgrade is breaking: deferred to another PR)
  • ci: updated linting rules, in line with other go-openapi repos

    • exception: disabled the "nilerr" linter, which currently crashes for this repo
  • ci:updated github actions, in line with other go-openapi repos

  • doc: updated README links and badges

  • major relinting of all test code

  • relinting non-test code is deferred to another PR (hopefully with less changes)

  • changes to non-test code are limited to:

    • addition of (temporary) nolint directives
    • gofmt/typos in comments
    • unused params

TODO in forthcoming PRs:

  • upgrade to yaml/v3
  • upgrade OTEL packages
  • gradually remove most "nolint" directives in non-test code
  • complete test refactoring: some nolint directives have been added to skip more complex refacoring (e.g. duplicate code)

* updated go version requirement to go1.19
* updated dependencies, excepted:
   * yaml (upgrade to v3 deferred to another PR)
   * otel (upgrade is breaking: deferred to another PR)

* ci: updated linting rules, in line with other go-openapi repos
   * exception: disabled the "nilerr" linter, which currently crashes for
     this repo
* ci:updated github actions, in line with other go-openapi repos
* doc: updated README links and badges

* major relinting of all test code
* relinting non-test code is deferred to another PR (hopefully with less changes)
* changes to non-test code are limited to:
  * addition of (temporary) nolint directives
  * gofmt/typos in comments
  * unused params

TODO in forthcoming PRs:
* [ ] upgrade to yaml/v3
* [ ] upgrade OTEL packages
* [ ] gradually remove most "nolint" directives in non-test code
* [ ] complete test refactoring: some nolint directives have been added
      to skip more complex refacoring (e.g. duplicate code)

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
Copy link

codecov bot commented Dec 7, 2023

Codecov Report

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

Comparison is base (1ade6d4) 80.26% compared to head (e4cc157) 80.26%.

Files Patch % Lines
client/request.go 80.00% 1 Missing ⚠️
client/runtime.go 80.00% 1 Missing ⚠️
middleware/context.go 80.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##           master     #269   +/-   ##
=======================================
  Coverage   80.26%   80.26%           
=======================================
  Files          44       44           
  Lines        3350     3350           
=======================================
  Hits         2689     2689           
  Misses        546      546           
  Partials      115      115           
Flag Coverage Δ
oldstable 80.26% <92.50%> (?)
stable 80.26% <92.50%> (?)

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
Copy link
Member Author

fredbi commented Dec 7, 2023

NOTE to reviewers

Ok so this one is huge. Apologies... That's because this PR has long been overdue...

To simplify a bit the review process, I've not addressed (yet) linting issues in non-testing code other
than adding nolint directives. So go files which are not test programs should normally only differ in their comments, or changes that can be verified at build time (e.g. unused parameter declaration, gofmt).

Hopefully this PR makes more readable to contributors the huge work that has been accumulated to test thoroughly this package.

@fredbi
Copy link
Member Author

fredbi commented Dec 7, 2023

Status of the above-mentioned forthcoming PRs:

@casualjim casualjim merged commit 381952b into go-openapi:master Dec 8, 2023
10 checks passed
@fredbi fredbi deleted the chore/update-linting branch December 8, 2023 08:43
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

2 participants