Skip to content

chore: relinted codebase#3

Merged
fredbi merged 1 commit intogo-openapi:masterfrom
fredbi:refactor
Mar 23, 2026
Merged

chore: relinted codebase#3
fredbi merged 1 commit intogo-openapi:masterfrom
fredbi:refactor

Conversation

@fredbi
Copy link
Member

@fredbi fredbi commented Mar 23, 2026

  • Exported API reduced to what go-swagger is calling: all other exported symbols are now unexported

  • carried out all simple relinting actions

  • modernized statements (use iterators, ...)

  • used nolint where appropriate for now, pending future deeper refactoring actions

  • refactored deeply nested structures and complex functions (including test code)

  • refactored code duplication

  • relaxed a few linting constraints (complexity, maintainability index)

No change in semantics, no change in test assertions

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

Short description

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.

* Exported API reduced to what go-swagger is calling: all other exported
  symbols are now unexported

* carried out all simple relinting actions
* modernized statements (use iterators, ...)
* used nolint where appropriate for now, pending future deeper
  refactoring actions
* refactored deeply nested structures and complex functions (including
  test code)
* refactored code duplication
* relaxed a few linting constraints (complexity, maintainability index)

No change in semantics, no change in test assertions

Signed-off-by: Frédéric BIDON <fredbi@yahoo.com>
@codecov
Copy link

codecov bot commented Mar 23, 2026

Codecov Report

❌ Patch coverage is 67.03854% with 325 lines in your changes missing coverage. Please review.
✅ Project coverage is 71.63%. Comparing base (e9cab73) to head (01925e5).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
schema.go 54.60% 106 Missing and 42 partials ⚠️
parser.go 80.89% 30 Missing and 17 partials ⚠️
parameters.go 59.09% 26 Missing and 10 partials ⚠️
responses.go 52.54% 18 Missing and 10 partials ⚠️
application.go 71.76% 15 Missing and 9 partials ⚠️
spec.go 0.00% 15 Missing and 2 partials ⚠️
taggers.go 82.95% 12 Missing and 3 partials ⚠️
assertions.go 0.00% 3 Missing ⚠️
operations.go 87.50% 3 Missing ⚠️
meta.go 0.00% 2 Missing ⚠️
... and 1 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master       #3      +/-   ##
==========================================
- Coverage   71.80%   71.63%   -0.17%     
==========================================
  Files          13       14       +1     
  Lines        3862     3794      -68     
==========================================
- Hits         2773     2718      -55     
+ Misses        785      761      -24     
- Partials      304      315      +11     

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

@fredbi fredbi merged commit 7a8145b into go-openapi:master Mar 23, 2026
20 of 21 checks passed
@fredbi fredbi deleted the refactor branch March 23, 2026 09:29
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.

1 participant