Repository navigation
Conversation
|
r? @scottmcm rustbot has assigned @scottmcm. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
cc @tgross35 |
This comment has been minimized.
This comment has been minimized.
|
Doing a run with tests in case somebody has a different value for @craterbot run mode=build-and-test |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
FCP started in #151073 |
|
@Apersoma please squash the commits and give it a reasonable message, the first commit just says "cmplt" (we don't have squash merging on this repo). |
|
🚧 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
🎉 Experiment
Footnotes
|
|
@craterbot run mode=build-and-test p=1 crates=https://crater-reports.s3.amazonaws.com/pr-154065/retry-regressed-list.txt |
|
👌 Experiment ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more |
|
Crater isn't too bad, but there are a couple of crates that I think we could resolve reasonably easy:
Crates that depend on Let's give some time for fixes here. @rustbot blocked |
|
Reminder, once the PR becomes ready for a review, use |
|
not me fat-fingering "unsubscribe". oops! |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
@Apersoma Are you able to update this PR, squashing, rebasing and fixing any issues? Otherwise as the author of the changes I'd be happy to open a PR to stabilise these changes. |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
|
I forgot no merge commits, goddamit. can someone help? I haven't really touched stuff since august for personal reasons but I completely forgot how to undo things that have already been committed. Also remember there being a lot of complications. |
|
Can you do the instructions at #154065 (comment)? Usually that should resolve things. If you get very stuck then let me know and I can push it for you. |
|
@Apersoma gentle ping for the rebase and commit message update |
24a4118 to
ddfecb8
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
|
The job Click to see the possible cause of the failure (guessed by this bot)Important For more information how to resolve CI failures of this job, visit this link. |
| // | ||
| #![allow(unstable_name_collisions)] // FIXME(float_bits_const): remove when stable |
There was a problem hiding this comment.
Could you update the comment to FIXME(float_masks)? Looks like that's the main conflict now.
|
☔ The latest upstream changes (presumably #165243) made this pull request unmergeable. Please resolve the merge conflicts by rebasing. |
View all comments
Tracking issue: #151073
Closes: #151073