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

refactor: migrate API specs to OpenAPI 3.1.0 ♻️ #12

Merged
merged 8 commits into from
Dec 27, 2024

Conversation

kareemmahlees
Copy link
Owner

  • Makes the code gen process part of the code utilizing go generate. Now, swagger docs are formatted and generated, and GraphQL code is generated by running go generate.
  • Uses scalar docs as the front end of swagger specs instead of the plain old swagger UI.

@kareemmahlees kareemmahlees merged commit dfecb91 into master Dec 27, 2024
3 checks passed
@kareemmahlees kareemmahlees deleted the refactor/code-generation branch December 27, 2024 15:44
Copy link

codecov bot commented Dec 27, 2024

Codecov Report

Attention: Patch coverage is 1.17647% with 84 lines in your changes missing coverage. Please review.

Project coverage is 13.52%. Comparing base (f621bea) to head (b78eb5e).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
cmd/gen/main.go 0.00% 40 Missing ⚠️
internal/graph/generated.go 0.00% 21 Missing ⚠️
internal/server.go 5.26% 16 Missing and 2 partials ⚠️
cmd/sqlite3.go 0.00% 2 Missing ⚠️
main.go 0.00% 2 Missing ⚠️
cmd/pg.go 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           master      #12       +/-   ##
===========================================
- Coverage   88.30%   13.52%   -74.79%     
===========================================
  Files          14       23        +9     
  Lines         727     4783     +4056     
===========================================
+ Hits          642      647        +5     
- Misses         58     4106     +4048     
- Partials       27       30        +3     

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

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