-
Notifications
You must be signed in to change notification settings - Fork 276
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
Use Coursier defaults, which also adds support for COURSIER_REPOSITORIES #1524
Use Coursier defaults, which also adds support for COURSIER_REPOSITORIES #1524
Conversation
42285dd
to
a5c102f
Compare
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.
LGTM 👍 Thank you!
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.
LGTM! thanks 🎉
) | ||
private def repositories: Array[Repository] = { | ||
// Default repositories are ivy2local, central and also anything in COURSIER_REPOSITORIES overrides | ||
Repository.defaults().asScala.toArray ++ Array( |
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.
Thanks @tanishiking ! We're super keen to use this, are there nightly builds still? and/or when do you next plan to release? Thanks both for the speedy review |
OK! Since this PR is merged, It would be good timing for releasing 2.1.1, and I'll release it now :) |
Ultimate speed 💪 - thanks! |
This is another crack at #1436 which I think is a bit easier now we've got the Coursier interface on board.
I believe that fetching Coursier's default repositories also prompts Couriser to check COURSIER_REPOSITORIES env var and
couriser.repositories
property - so by using defaults here we get COURSIER_REPOSITORIES compatibilityPreviously the default repositories were being totally overwritten by the hard coded repository list
I believe this fulfils #1521