Skip to content

Ignore test fix_in_dependency to make a change in rustc#13218

Closed
petrochenkov wants to merge 1 commit intorust-lang:masterfrom
petrochenkov:ignorefix
Closed

Ignore test fix_in_dependency to make a change in rustc#13218
petrochenkov wants to merge 1 commit intorust-lang:masterfrom
petrochenkov:ignorefix

Conversation

@petrochenkov
Copy link
Copy Markdown
Contributor

This test relies on some specific diagnostics being produced in rustc, but those diagnostics are currently going through changes (rust-lang/rust#119204 and there will be some more follow up PRs).

@rustbot
Copy link
Copy Markdown
Collaborator

rustbot commented Dec 29, 2023

r? @epage

(rustbot has picked a reviewer for you, use r? to override)

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 29, 2023
@petrochenkov
Copy link
Copy Markdown
Contributor Author

The CI failure looks unrelated.

@ehuss
Copy link
Copy Markdown
Contributor

ehuss commented Dec 29, 2023

I can post a PR to fix it to not rely on the compiler.

@ehuss
Copy link
Copy Markdown
Contributor

ehuss commented Dec 29, 2023

Posted #13220 as an alternative.

@bors
Copy link
Copy Markdown
Contributor

bors commented Jan 1, 2024

☔ The latest upstream changes (presumably #13220) made this pull request unmergeable. Please resolve the merge conflicts.

@ehuss
Copy link
Copy Markdown
Contributor

ehuss commented Jan 2, 2024

Closing as this should be resolved by #13220. Thanks!

@ehuss ehuss closed this Jan 2, 2024
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