Skip to content
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

fix: Fix formatting requests hanging when r-a is still starting #13428

Merged
merged 1 commit into from
Oct 17, 2022

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Oct 17, 2022

The reason for that was that we were calculating the crate defmaps of the file we are saving causing us to get stuck waiting on their expensive computation, while we only need the relevant crate ids to fetch the edition of the crates.

Closes #4054
Closes #11654

@Veykril
Copy link
Member Author

Veykril commented Oct 17, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Oct 17, 2022

📌 Commit 359836d has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 17, 2022

⌛ Testing commit 359836d with merge 89ca0dc...

bors added a commit that referenced this pull request Oct 17, 2022
fix: Fix formatting requests hanging when r-a is still starting

The reason for that was that we were calculating the crate defmaps of the file we are saving by accident causing us to get stuck waiting on their expensive computation, while we only need the relevant crate id.

Closes #4054
Closes #11654
@bors
Copy link
Contributor

bors commented Oct 17, 2022

💔 Test failed - checks-actions

The reason for that was that we were calculating the crate defmaps
of the file we are saving by accident causing us to get stuck waiting
on their expensive computation, while we only need the relevant crate
id.
@Veykril
Copy link
Member Author

Veykril commented Oct 17, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Oct 17, 2022

📌 Commit a762bac has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Oct 17, 2022

⌛ Testing commit a762bac with merge 4d4c05d...

@bors
Copy link
Contributor

bors commented Oct 17, 2022

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 4d4c05d to master...

@bors bors merged commit 4d4c05d into rust-lang:master Oct 17, 2022
@Veykril Veykril deleted the fmt-stuck branch October 18, 2022 13:23
bors added a commit that referenced this pull request Oct 20, 2022
fix: Fix DidSaveDocument requests blocking the server on startup

Follow up to #13428
Fixes #13434
bors added a commit that referenced this pull request Oct 20, 2022
fix: Fix DidSaveDocument requests blocking the server on startup

Follow up to #13428
Fixes #13434
bors added a commit that referenced this pull request Oct 20, 2022
fix: Fix DidSaveDocument requests blocking the server on startup

Follow up to #13428
Fixes #13434
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Format on save is delayed rustfmt on save takes unusually long
2 participants