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

Add regression test #15025

Merged
merged 1 commit into from
May 5, 2022
Merged

Add regression test #15025

merged 1 commit into from
May 5, 2022

Conversation

nicolasstucki
Copy link
Contributor

Closes #9685

@anatoliykmetyuk anatoliykmetyuk merged commit 84dd54d into scala:main May 5, 2022
@anatoliykmetyuk anatoliykmetyuk deleted the close-9685 branch May 5, 2022 09:46
@tgodzik
Copy link
Contributor

tgodzik commented Feb 2, 2023

Are we sure this one got fixed? There is still an open issue about it scalameta/munit#626 <- is there anything munit needs to change?

@nicolasstucki
Copy link
Contributor Author

The self-contained minimization started working. It is possible that the original example hit the bug differently. I suggest re-minimizing the issue to understand what is failing.

@tgodzik
Copy link
Contributor

tgodzik commented Feb 2, 2023

There is a minimization in the issue, also pasted it into original issue.

@julienrf
Copy link
Contributor

It is not clear to me why this test passes. I’ve pushed a very slight variation that fails here: 1e7bf82

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Confusing suggested import involving inline implicit
4 participants