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

Uncomment test for #8734 #9071

Merged
merged 1 commit into from
Jun 30, 2022
Merged

Uncomment test for #8734 #9071

merged 1 commit into from
Jun 30, 2022

Conversation

Alexendoo
Copy link
Member

I believe the issue was an interaction between rustfix and span_lint_and_sugg_for_edges, so this would've been fixed by rust-lang/rust#98261 (Thanks, @WaffleLapkin!)

Closes #8734

changelog: none

@rust-highfive
Copy link

r? @dswij

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Jun 30, 2022
Copy link
Member

@dswij dswij left a comment

Choose a reason for hiding this comment

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

Thanks for this! (and also @WaffleLapkin for the sweet fix)

@dswij
Copy link
Member

dswij commented Jun 30, 2022

@bors r+

@bors
Copy link
Contributor

bors commented Jun 30, 2022

📌 Commit 6c61f71 has been approved by dswij

@bors
Copy link
Contributor

bors commented Jun 30, 2022

⌛ Testing commit 6c61f71 with merge 4198013...

@bors
Copy link
Contributor

bors commented Jun 30, 2022

☀️ Test successful - checks-action_dev_test, checks-action_remark_test, checks-action_test
Approved by: dswij
Pushing 4198013 to master...

@bors bors merged commit 4198013 into rust-lang:master Jun 30, 2022
@Alexendoo Alexendoo deleted the 8734 branch June 30, 2022 11:03
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.

Clippy --fix panics with index error
4 participants