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

Ignore test fix_in_dependency to make a change in rustc #13218

Closed
wants to merge 1 commit into from

Conversation

petrochenkov
Copy link
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
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
Contributor Author

The CI failure looks unrelated.

@ehuss
Copy link
Contributor

ehuss commented Dec 29, 2023

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

@ehuss
Copy link
Contributor

ehuss commented Dec 29, 2023

Posted #13220 as an alternative.

@bors
Copy link
Collaborator

bors commented Jan 1, 2024

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

@ehuss
Copy link
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