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

🎨 Style: Add inamedparam linter #2848

Merged
merged 1 commit into from
Feb 10, 2024

Conversation

nickajacks1
Copy link
Member

Description

inamedparam enforces that parameters in interface definitions be named. This is important for clarity so that users and implementers can easily understand the purpose of each parameter.

Related to #2817

Type of Change

  • Code consistency (non-breaking change which improves code reliability and robustness)

Checklist

Before you submit your pull request, please make sure you meet these requirements:

  • Conducted a self-review of the code and provided comments for complex or critical parts.
  • Updated the documentation in the /docs/ directory for Fiber's documentation.
  • Ensured that new and existing unit tests pass locally with the changes.

inamedparam enforces that parameters in interface definitions be named.
This is important for clarity so that users and implementers can easily
understand the purpose of each parameter.
@ReneWerner87 ReneWerner87 merged commit 92dd8d6 into gofiber:main Feb 10, 2024
15 checks passed
nickajacks1 added a commit to nickajacks1/fiber that referenced this pull request Feb 11, 2024
efectn pushed a commit that referenced this pull request Feb 11, 2024
* 📚 Docs: Update version numbers for v3

* 📚 Docs: Update template docs to reflect changes in #2848

---------

Co-authored-by: Juan Calderon-Perez <835733+gaby@users.noreply.github.com>
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.

2 participants