-
-
Notifications
You must be signed in to change notification settings - Fork 14.4k
Remove suggestion from importing unstable items on stable channel #149440
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? @fee1-dead rustbot has assigned @fee1-dead. Use |
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.
r=me after addressing nit
| @@ -0,0 +1,9 @@ | |||
| //@ only-nightly | |||
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.
Is it possible to use revisions here to avoid duplicating the same file?
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.
5b00345 to
34c80e4
Compare
|
@bors r=fee1-dead |
Remove suggestion from importing unstable items on stable channel Fixes #149402
Remove suggestion from importing unstable items on stable channel Fixes #149402
This comment has been minimized.
This comment has been minimized.
|
💔 Test failed - checks-actions |
|
@jieyouxu I build a stage2 on local machine, and found out the From the code, seems we will set |
This comment was marked as outdated.
This comment was marked as outdated.
|
Ok, it looks like One option you may have is to write this as a run-make test, with two stderr tracked: a stable rustc, and an unstable rustc. Gate the run-make itself using
|
|
@rustbot author |
|
@bors r- (sync) |
34c80e4 to
1aac0b4
Compare
Remove suggestion from importing unstable items on stable channel Fixes rust-lang#149402
Rollup of 9 pull requests Successful merges: - #148769 (Stabilize `alloc_layout_extra`) - #149440 (Remove suggestion from importing unstable items on stable channel) - #150200 (Add title field to `ice.md` issue template) - #150955 (Underscore-prefixed bindings are explicitly allowed to be unused) - #151200 (time: Add saturating arithmetic for `SystemTime`) - #151235 (Change field `bit_width: usize` to `bits: u32` in type info) - #151242 (Port #[needs_allocator] to attribute parser) - #151274 (Include a link to `count_ones` in the docs for `uN::count_zeros` [docs only]) - #151279 (remove trailing periods in built-in attribute gate messages) r? @ghost
|
Commit e1aee29 has been unapproved. |
e1aee29 to
c578560
Compare
|
@bors try jobs=dist-various-1 |
This comment has been minimized.
This comment has been minimized.
Remove suggestion from importing unstable items on stable channel try-job: dist-various-1
|
@rustbot ready |
|
@bors r+ rollup |
This comment has been minimized.
This comment has been minimized.
What is this?This is an experimental post-merge analysis report that shows differences in test outcomes between the merged PR and its parent PR.Comparing 0f14563 (parent) -> 39052da (this PR) Test differencesShow 23 test diffsStage 1
Stage 2
Additionally, 19 doctest diffs were found. These are ignored, as they are noisy. Job group index
Test dashboardRun cargo run --manifest-path src/ci/citool/Cargo.toml -- \
test-dashboard 39052daf937d46373ac29778e1b8853c52c0cc25 --output-dir test-dashboardAnd then open Job duration changes
How to interpret the job duration changes?Job durations can vary a lot, based on the actual runner instance |
|
Finished benchmarking commit (39052da): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results (primary 0.1%, secondary 2.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
CyclesResults (primary -2.4%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Binary sizeResults (primary 0.0%)A less reliable metric. May be of interest, but not used to determine the overall result above.
Bootstrap: 475.466s -> 473.794s (-0.35%) |
Fixes #149402