Skip to content

feat: Load proc-macros asynchronously #14405

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

Merged
merged 4 commits into from
Mar 25, 2023
Merged

Conversation

Veykril
Copy link
Member

@Veykril Veykril commented Mar 25, 2023

Closes #8646

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Mar 25, 2023
@Veykril Veykril force-pushed the proc-macro-loading branch from 5e35334 to e9fb2ff Compare March 25, 2023 15:50
@Veykril Veykril marked this pull request as ready for review March 25, 2023 15:50
@Veykril Veykril changed the title Split out proc-macros from the CrateGraph internal: Split out proc-macros from the CrateGraph Mar 25, 2023
@Veykril
Copy link
Member Author

Veykril commented Mar 25, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Mar 25, 2023

📌 Commit e9fb2ff has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Mar 25, 2023

⌛ Testing commit e9fb2ff with merge a44f244...

bors added a commit that referenced this pull request Mar 25, 2023
internal: Split out proc-macros from the CrateGraph

cc #8646
@bors
Copy link
Contributor

bors commented Mar 25, 2023

💔 Test failed - checks-actions

@Veykril Veykril changed the title internal: Split out proc-macros from the CrateGraph feat: Load proc-macros asynchronously Mar 25, 2023
@Veykril
Copy link
Member Author

Veykril commented Mar 25, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Mar 25, 2023

📌 Commit e5f24a6 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Mar 25, 2023

⌛ Testing commit e5f24a6 with merge 8433407...

bors added a commit that referenced this pull request Mar 25, 2023
feat: Load proc-macros asynchronously

Closes #8646
@Veykril
Copy link
Member Author

Veykril commented Mar 25, 2023

@bors r+

@bors
Copy link
Contributor

bors commented Mar 25, 2023

💡 This pull request was already approved, no need to approve it again.

  • This pull request is currently being tested. If there's no response from the continuous integration service, you may use retry to trigger a build again.

@bors
Copy link
Contributor

bors commented Mar 25, 2023

📌 Commit e5f24a6 has been approved by Veykril

It is now in the queue for this repository.

@bors
Copy link
Contributor

bors commented Mar 25, 2023

⌛ Testing commit e5f24a6 with merge 90340b7...

@bors
Copy link
Contributor

bors commented Mar 25, 2023

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 90340b7 to master...

1 similar comment
@bors
Copy link
Contributor

bors commented Mar 25, 2023

☀️ Test successful - checks-actions
Approved by: Veykril
Pushing 90340b7 to master...

@bors
Copy link
Contributor

bors commented Mar 25, 2023

👀 Test was successful, but fast-forwarding failed: 422 Changes must be made through a pull request.

@bors bors merged commit 90340b7 into rust-lang:master Mar 25, 2023
@@ -1,5 +1,5 @@
<!---
lsp_ext.rs hash: 37f31ae648632897
lsp_ext.rs hash: 92fe1037312754df
Copy link
Member

Choose a reason for hiding this comment

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

Might want to add a new command here!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
S-waiting-on-review Status: Awaiting review from the assignee but also interested parties.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Load procedural macros asynchronously
4 participants