Skip to content

fix: fix problem in swagger file due to pr merge and regenerate oapi-codegen files & add dev url#1383

Merged
Eraxyso merged 4 commits intofix/openapifrom
fix/openapi-fix-openapi
Jan 31, 2026
Merged

fix: fix problem in swagger file due to pr merge and regenerate oapi-codegen files & add dev url#1383
Eraxyso merged 4 commits intofix/openapifrom
fix/openapi-fix-openapi

Conversation

@Eraxyso
Copy link
Copy Markdown
Contributor

@Eraxyso Eraxyso commented Sep 2, 2025

No description provided.

@codecov
Copy link
Copy Markdown

codecov Bot commented Sep 2, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 66.08%. Comparing base (37c88b9) to head (b4f1aeb).
⚠️ Report is 53 commits behind head on fix/openapi.

Additional details and impacted files
@@               Coverage Diff               @@
##           fix/openapi    #1383      +/-   ##
===============================================
+ Coverage        62.12%   66.08%   +3.96%     
===============================================
  Files               25       25              
  Lines             4161     3468     -693     
===============================================
- Hits              2585     2292     -293     
+ Misses            1231      831     -400     
  Partials           345      345              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@Eraxyso Eraxyso requested a review from kaitoyama September 2, 2025 14:52
@Eraxyso Eraxyso requested a review from Copilot October 14, 2025 08:35
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR fixes issues in the swagger file that occurred due to a PR merge and regenerates the oapi-codegen files with the updated version.

  • Updated oapi-codegen version from v2.4.1 to v2.5.0 in generated Go files
  • Removed duplicate/incorrect schema definitions from swagger.yaml
  • Regenerated the embedded swagger specification in Go code

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

File Description
openapi/types.go Updated code generation comment to reflect v2.5.0
openapi/spec.go Updated code generation comment and regenerated embedded swagger spec
openapi/server.go Updated code generation comment to reflect v2.5.0
docs/swagger/swagger.yaml Removed duplicate EditResponse schema and incorrect allOf block in ResponsesWithInfo

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@Eraxyso Eraxyso force-pushed the fix/openapi-fix-openapi branch from da1ac74 to e9edf3e Compare January 31, 2026 05:46
@Eraxyso Eraxyso changed the title fix: fix problem in swagger file due to pr merge and regenerate oapi-codegen files fix: fix problem in swagger file due to pr merge and regenerate oapi-codegen files & add dev url Jan 31, 2026
Copy link
Copy Markdown
Contributor

@kaitoyama kaitoyama left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Eraxyso Eraxyso merged commit a3a1b55 into fix/openapi Jan 31, 2026
9 checks passed
@Eraxyso Eraxyso deleted the fix/openapi-fix-openapi branch January 31, 2026 05:57
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.

3 participants