Skip to content

Conversation

@GrigorenkoPV
Copy link
Contributor

@GrigorenkoPV GrigorenkoPV commented Nov 6, 2025

Split from #148590

Tracking issue for update and try_update: #135894

@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 6, 2025
@rustbot
Copy link
Collaborator

rustbot commented Nov 6, 2025

r? @JonathanBrouwer

rustbot has assigned @JonathanBrouwer.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Copy link
Member

@Kivooeo Kivooeo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, just adds update and try_update checks the same way it was for fetch_update

One small nit:

View changes since this review

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you please update test name, since its now not only fetch_update test or create another test for this feature

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I decided to not separate this into separate files, because it would make it more difficult to keep the tests in sync.

I tried to maybe use revisions approach, but it would still lead to each error message being written 3 times, and wouldn't really simplify the code that much.

I decided to just rename it from "fetch-update" to simply "update" as it is the word that all 3 methods have in common.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yeah, that's fine, let's wait on CI green

@Kivooeo Kivooeo assigned Kivooeo and unassigned JonathanBrouwer Nov 6, 2025
@GrigorenkoPV GrigorenkoPV force-pushed the invalid_atomic_ordering branch from fbac049 to bd23d55 Compare November 7, 2025 13:17
@GrigorenkoPV GrigorenkoPV requested a review from Kivooeo November 7, 2025 13:24
@Kivooeo
Copy link
Member

Kivooeo commented Nov 7, 2025

@bors r+

@bors
Copy link
Collaborator

bors commented Nov 7, 2025

📌 Commit bd23d55 has been approved by Kivooeo

It is now in the queue for this repository.

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Nov 7, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. 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.

5 participants