-
Notifications
You must be signed in to change notification settings - Fork 39
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
core, editoast, front: stop using speed-limit tags' long names #8417
Conversation
Codecov ReportAttention: Patch coverage is
❗ Your organization needs to install the Codecov GitHub app to enable full functionality. Additional details and impacted files@@ Coverage Diff @@
## dev #8417 +/- ##
============================================
+ Coverage 37.03% 37.06% +0.03%
+ Complexity 2212 2209 -3
============================================
Files 1253 1255 +2
Lines 113882 114079 +197
Branches 3188 3182 -6
============================================
+ Hits 42176 42286 +110
- Misses 69810 69901 +91
+ Partials 1896 1892 -4
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
fe171cd
to
1951daa
Compare
03c1305
to
7e4bb8f
Compare
7e4bb8f
to
bdf7d07
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 for tests/
Fixed integration tests in ddc2d2f (@shenriotpro may re-review) |
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 job!
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
ddc2d2f
to
fbb19a4
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 for e2e tests
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 (tested with new dump)
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 :)
* stop fallback on long-names (core) * remove short-id extractors (front) * adapt examples and small_infra also remove default-speed, as it is unused Signed-off-by: Pierre-Etienne Bougué <[email protected]>
fbb19a4
to
6caffd5
Compare
also remove default-speed, as it is unused
Resolves #7977
🔍
This PR can be reviewed, but not merged👇TODO: