Skip to content

Conversation

bm-w
Copy link
Contributor

@bm-w bm-w commented May 18, 2023

Fixes exit/yield points not getting highlighted in such blocks for highlight_related (#14813; and possibly other bugs in features that use preorder_expr).

Fixes exit/yield points not getting highlighted in such blocks for `highlight_related` (rust-lang#14813; and possibly other bugs in features that use `preorder_expr`).
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label May 18, 2023
Co-authored-by: Bastiaan Marinus van de Weerd <[email protected]>
@Veykril
Copy link
Member

Veykril commented May 19, 2023

Thanks!
@bors r+

@bors
Copy link
Contributor

bors commented May 19, 2023

📌 Commit c0519da has been approved by Veykril

It is now in the queue for this repository.

@Veykril Veykril changed the title Fix preorder_expr skipping the else block of let-else statements fix: Fix preorder_expr skipping the else block of let-else statements May 19, 2023
@bors
Copy link
Contributor

bors commented May 19, 2023

⌛ Testing commit c0519da with merge e8ae2d3...

@bors
Copy link
Contributor

bors commented May 19, 2023

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

1 similar comment
@bors
Copy link
Contributor

bors commented May 19, 2023

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

@bors
Copy link
Contributor

bors commented May 19, 2023

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

@bors bors merged commit e8ae2d3 into rust-lang:master May 19, 2023
@bm-w bm-w deleted the fix/highlight-let-else-return branch May 19, 2023 15:14
@lnicola
Copy link
Member

lnicola commented May 20, 2023

changelog fix (first contribution) traverse else blocks of let-else in preorder_expr

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.

5 participants