-
Notifications
You must be signed in to change notification settings - Fork 8.1k
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
fix(router): tree bug where loop index is not decremented. #3460
Conversation
Codecov Report
@@ Coverage Diff @@
## master #3460 +/- ##
=======================================
Coverage 98.63% 98.63%
=======================================
Files 42 42
Lines 3140 3140
=======================================
Hits 3097 3097
Misses 29 29
Partials 14 14
Flags with carried forward coverage won't be shown. Click here to find out more.
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
f4dfe65
to
3bda7d8
Compare
Also add test to validate fix. fixes gin-gonic#3459 Signed-off-by: Kristian Svalland <kristian.svalland@gmail.com>
3bda7d8
to
fd8bbdc
Compare
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, thanks!
Also add test to validate fix.
fixes #3459