-
Notifications
You must be signed in to change notification settings - Fork 1.1k
fix-#9026 eliminate .. in paths #9027
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
You can edit .github/workflows/ci.yaml, but if you tested it manually that should be good enough |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm not sure I understand what changed and why the change was necessary, can you write a more detailed commit message, and add a documentaiton comment above common
?
Looks like a transient failure in the community build:
@anatoliykmetyuk It looks like sourcecode is downloading stuff (mill itself?) in the community build and this isn't cached, that might explain some of the network consumption reported by Fabien if all the mill projects do this. |
Will first nightly be tonight? |
There was one last night already: https://repo1.maven.org/maven2/ch/epfl/lamp/dotty_0.25/0.25.0-bin-20200523-5358651-NIGHTLY/ |
Awesome! |
not sure how to ensure we test on 11 in the PR