Skip to content

Conversation

nicolasstucki
Copy link
Contributor

tests/neg/missing-implicit1.scala can easily have a different (but correct)
output when the CI is overloaded.

`tests/neg/missing-implicit1.scala` can easily have a different (but correct)
output when the CI is overloaded.
@nicolasstucki
Copy link
Contributor Author

@nicolasstucki nicolasstucki marked this pull request as ready for review June 17, 2020 16:11
@nicolasstucki nicolasstucki added the fasttrack Simple fix. Reviewer should merge or apply additional changes directly. label Jun 17, 2020
@odersky
Copy link
Contributor

odersky commented Jun 18, 2020

The errors arise since now the import suggestion budget is global, not per suggestion, and missing-implicits1 has many errors where a search is needed. #9189 splits missing-implicits1 into several tests. I think that's a better solution, since it avoids overly long search times for other tests with unrelated errors. But we should merge this quickly, so that the flakeyness of testing goes away. @nicolasstucki I have made you a reviewer of that one.

@odersky odersky closed this Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fasttrack Simple fix. Reviewer should merge or apply additional changes directly.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants