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 rule syntax version for all internal routers #10689
Fix rule syntax version for all internal routers #10689
Conversation
8108e61
to
5027714
Compare
I previously added explicit But as it turns out this would probably break the "workaround" for #10672. Therefore I reverted this change for now and only fixed the redirection. The problem I tried to fix there was that |
5027714
to
19bad2b
Compare
19bad2b
to
f9fd3d1
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.
Thanks!
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
f9fd3d1
to
26b85bd
Compare
You don't need to rebase, our bot will handle the rebase when the PR has 3 approvals. |
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.
Thanks 👍
Co-authored-by: romain <rtribotte@users.noreply.github.com>
26b85bd
to
0a75545
Compare
What does this PR do?
This PR sets the rule syntax to
v3
explicitly fordynamic.Router
initialization for all internal routes.Motivation
Fixes #10688