Closed uvulpos closed 4 months ago
[!WARNING]
Rate limit exceeded
@uvulpos has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 19 minutes and 20 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 74ce35d3e3deebc9b48dd9927aa117e8737bcdec and d373a9a3edb0c632bd9afd9cd96c77e27c065d05.
The recent updates introduce Swagger and Swagger UI setup instructions for a Go project, explaining the installation of Swag, generating documentation, and configuring the Swagger UI. Additionally, they modify existing code to embed Swagger UI files into the Go application. These changes enhance the ability to document and visualize APIs within the application.
Files | Change Summary |
---|---|
README.md |
Added instructions for setting up and accessing Swagger UI, including installing and running Swag. |
files.go |
Introduced functionality to embed Swagger UI files through an embedded file system. |
swagger.go |
Updated references from swaggerFiles to SwaggerFileSystem and modified import statements. |
🌟 Amidst the codes where logic resides,
Swagger now flows, with joy it presides.
Documentation blooms, a map so clear,
Through embedded files, it's all right here.
Visions of APIs, in a UI so grand,
Celebrate the change, in this code-filled land. 🚀
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
@uvulpos The dependency is not unmaintained, probably worth doing a PR there instead of merging all this code here.
@gaby I made a ping comment to the abandoned pr to merge a newer version see here. Also there was just one feature pr merged last year
And also by merging it to our own code, we could solve the google font api request. Should be fine in most countries, but in europe this could be an issue. There were incidents where companies and individual website hosts got fined for requesting google fonts before asking because of the GDPR. This way it would be more safety regarding legal issues
Since swaggo/files updated their repo, this is obsolet
The upstream dependency https://github.com/swaggo/files seems unmaintained so I decided to bundle it directly into this gofiber packages so people can use newer versions of swagger.
Origin repo uses a submodule for this, I'm not too familiar with submodules and to automate this process, but this could be a good enhancement to run this periodically via github actions.
I ran the tests command but please test it yourself again please!
fixes #91 fixes #51
Summary by CodeRabbit
Documentation
New Features
Refactor
SwaggerFileSystem
.