Skip to content

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

Merged
bors merged 1 commit into
rust-lang:masterfrom
Veykril:fmt-stuck
Oct 17, 2022
Merged

fix: Fix formatting requests hanging when r-a is still starting#13428
bors merged 1 commit into
rust-lang:masterfrom
Veykril:fmt-stuck

Conversation

@Veykril

@Veykril Veykril commented Oct 17, 2022

Copy link
Copy Markdown
Member

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

Veykril commented Oct 17, 2022

Copy link
Copy Markdown
Member Author

@bors r+

@bors

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

📌 Commit 359836d has been approved by Veykril

It is now in the queue for this repository.

@bors

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

⌛ 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

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

💔 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

Veykril commented Oct 17, 2022

Copy link
Copy Markdown
Member Author

@bors r+

@bors

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

📌 Commit a762bac has been approved by Veykril

It is now in the queue for this repository.

@bors

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

⌛ Testing commit a762bac with merge 4d4c05d...

@bors

bors commented Oct 17, 2022

Copy link
Copy Markdown
Contributor

☀️ 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