-
Notifications
You must be signed in to change notification settings - Fork 5
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
feat: fiber support filter tag #38
Conversation
# Conflicts: # go.mod # go.sum
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #38 +/- ##
==========================================
+ Coverage 77.81% 78.46% +0.64%
==========================================
Files 10 10
Lines 861 873 +12
==========================================
+ Hits 670 685 +15
Misses 161 161
+ Partials 30 27 -3 ☔ View full report in Codecov by Sentry. |
# Conflicts: # go.mod # go.sum
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work! could you please also add this feature for Gin?
Co-authored-by: Wenbo Han <hwbrzzl@gmail.com>
# Conflicts: # go.mod # go.sum
Added |
# Conflicts: # go.mod # go.sum
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, could you please list all supported filter rules in the PR description?
for k, v := range r.instance.AllParams() { | ||
mu.Lock() | ||
data[k] = v | ||
mu.Unlock() | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch!
Closes goravel/goravel#304
Closes goravel/goravel#259
Closes goravel/goravel#247
Closes goravel/goravel#313
📑 Description
✅ Checks
ℹ Additional Information