-
Notifications
You must be signed in to change notification settings - Fork 8k
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: remove route in router #3145
base: master
Are you sure you want to change the base?
Conversation
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.
ok
@appleboy please review this pr. thanks . |
} | ||
|
||
} | ||
|
||
// Routes returns a slice of registered routes, including some useful information, such as: |
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.
missing all TODO tasks?
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.
what do you mean? do I need to create "task" for each TODO ?
maybe we should separate one gin-contrib, |
finish TODO
@appleboy finish TODO 。 @thinkerou It is all middleware in gin-contrib. you think the "REMOVE" function should be there? |
finish TODO
feature: 'remove' is complex feature . This submit only gives the basic framework, and relevant details need to be improved later, such as modifying the priority of nodes
test:already make the basic test
ps:
this is my first time to submit PR to gin,I have great passion to make contribute。please tell if I did not go the right way