Skip to content

Ignore test fix_in_dependency to make a change in rustc#13218

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

Ignore test fix_in_dependency to make a change in rustc#13218
petrochenkov wants to merge 1 commit into
rust-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

rustbot commented Dec 29, 2023

Copy link
Copy Markdown
Collaborator

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

ehuss commented Dec 29, 2023

Copy link
Copy Markdown
Contributor

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

@ehuss

ehuss commented Dec 29, 2023

Copy link
Copy Markdown
Contributor

Posted #13220 as an alternative.

@bors

bors commented Jan 1, 2024

Copy link
Copy Markdown
Contributor

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

@ehuss

ehuss commented Jan 2, 2024

Copy link
Copy Markdown
Contributor

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