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

Fixing a mistake of the comment in the limiter package #1790

Merged
merged 1 commit into from Feb 20, 2022

Conversation

qracer
Copy link

@qracer qracer commented Feb 20, 2022

closes #1789

@welcome
Copy link

welcome bot commented Feb 20, 2022

Thanks for opening this pull request! 🎉 Please check out our contributing guidelines. If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@efectn
Copy link
Member

efectn commented Feb 20, 2022

Thanks. Can you create same PR for https://github.com/gofiber/docs/blob/master/api/middleware/limiter.md?

@efectn efectn merged commit c5f11cc into gofiber:master Feb 20, 2022
@welcome
Copy link

welcome bot commented Feb 20, 2022

Congrats on merging your first pull request! 🎉 We here at Fiber are proud of you! If you need help or want to chat with us, join us on Discord https://gofiber.io/discord

@qracer qracer deleted the docfix branch February 20, 2022 16:14
@qracer
Copy link
Author

qracer commented Feb 20, 2022

Created PR for docs too

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

📚 A mistake in the limiter middleware source code comment
3 participants