-
Notifications
You must be signed in to change notification settings - Fork 1.1k
merge variances and Variances #1193
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
Conversation
Review by @sstucki |
Maybe this also caused the weird unpickler bug? |
Looks like your commit does not contain the deletion of Variances.scala
Unlikely, given that the bug appeared in the |
@odersky, I don't see how it would make a different test, with a different source path, fail. |
#1189 PR wasn't rebased over master. |
I've pushed a commit that fixes build. |
Same heisenbug as in #1190. I wasn't able to reproduce it locally and it's not always reproducible on jenkins. I'm going to go forward and merge this PR, as it fixes the development environment.
|
Both variances.scala and |
These files were supposed to be remove, their content are now in variances.scala
Fixed in 80b1247 |
LGTM. Thanks @smarter and sorry for breaking the build everyone. |
We had variances.scala and Variances.scala in pos and neg. This messes up MacOS royally, to a point where a rebase gets stuck in an infinite loop. I merged the two pairs of tests.