Skip to content

Disarm time 💣 (diagnostics) #133245

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed
wants to merge 1 commit into from

Conversation

jieyouxu
Copy link
Member

@jieyouxu jieyouxu commented Nov 20, 2024

Closes #129461.

Manually reverts Emit specific message for time<=0.3.35 #129343 to defuse a time 💣.

@jieyouxu jieyouxu added the S-blocked Status: Blocked on something else such as an RFC or other implementation work. label Nov 20, 2024
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Nov 20, 2024
@jieyouxu jieyouxu removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Nov 20, 2024
@apiraino
Copy link
Contributor

now we need a second timer to remind us to merge this 😄

@bors

This comment was marked as off-topic.

@jieyouxu jieyouxu force-pushed the disarm-time-bomb branch 2 times, most recently from 61c02cb to 2f2fc50 Compare March 8, 2025 01:45
@jieyouxu jieyouxu changed the title [IT'S NOT TIME YET] Disarm time bomb (diagnostics) [IT'S NOT TIME YET] Disarm time 💣 (diagnostics) Mar 8, 2025
@bors
Copy link
Collaborator

bors commented Mar 15, 2025

☔ The latest upstream changes (presumably #138464) made this pull request unmergeable. Please resolve the merge conflicts.

@bors
Copy link
Collaborator

bors commented Apr 30, 2025

☔ The latest upstream changes (presumably #140520) made this pull request unmergeable. Please resolve the merge conflicts.

Manually revert rust-lang#129343 `time` diagnostics, which is a temporary
diagnostic to mitigate inference breakage fallout.
@jieyouxu jieyouxu force-pushed the disarm-time-bomb branch from 9a6a705 to e713a31 Compare May 9, 2025 10:08
@jieyouxu jieyouxu changed the title [IT'S NOT TIME YET] Disarm time 💣 (diagnostics) Disarm time 💣 (diagnostics) May 9, 2025
@jieyouxu
Copy link
Member Author

jieyouxu commented May 9, 2025

(Already rolled into #140839)

@jieyouxu jieyouxu closed this May 9, 2025
@jieyouxu jieyouxu deleted the disarm-time-bomb branch May 9, 2025 10:12
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
S-blocked Status: Blocked on something else such as an RFC or other implementation work. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove time-boxed time inference regression mitigation note
4 participants