-
Notifications
You must be signed in to change notification settings - Fork 13.3k
Omit integer suffix when unnecessary #21710
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
Conversation
r? @nick29581 (rust_highfive has picked a reviewer for you, use r? to override) |
@bors: r+ ad20f5ae444afba3792ee15c6d59dc4b2e64925c |
Needs a rebase |
See PR # 21378 for context
Ping @frewsxcv |
@Alfie I unfortunately do not have reviewing rights :P You might need @nick29581 for that |
Sorry @frewsxcv :) |
Sorry, I didn't realise force push didn't notify. |
Conflicts: src/libsyntax/parse/parser.rs
⌛ Testing commit 9683745 with merge 2b0fd75... |
💔 Test failed - auto-linux-64-x-android-t |
@bors: retry |
@alexcrichton any idea why homu stopped? |
Ah I canceled it manually because this is included in the rollup (#21872) and wanted to put the rollup back on bors. |
Ahh cool, np :) |
See PR # 21378 for context