Skip to content
This repository was archived by the owner on Aug 18, 2020. It is now read-only.

Support JCenter in Bootstrap Launcher #10

Merged
merged 4 commits into from
May 25, 2019
Merged

Support JCenter in Bootstrap Launcher #10

merged 4 commits into from
May 25, 2019

Conversation

hlxid
Copy link
Member

@hlxid hlxid commented May 23, 2019

The Bootstrap Launcher was designed to download anything from Maven Central.

But since commit 97e67cb has added JDA for Discord the Bootstrap Launcher was not able to download it because it along with some of its dependencies are only hosted on JCenter.

This pr adds the ability to resolve dependencies from other repos, but currently only JCenter and Maven Central are used. It still tries to get anything from Maven Central and only uses JCenter, or any other repo that may be used in the future, when Maven Central doesn't have a dependency.

Any feedback would be appreciated.

hlxid added 3 commits May 19, 2019 16:06
The Bootstrap Launcher was designed to download anything from Maven Central.
But since commit 97e67cb has added JDA for Discord the Bootstrap Launcher
was not able to download it because it along with some of its dependencies are only hosted on JCenter.
This commit adds the ability to resolve dependencies from other repos,
 but currently only JCenter and Maven Central are used.
It still tries to get anything from Maven Central and only uses JCenter,
 or any other repo that may be used in the future, as a fallback
 if Maven Central doesn't have a dependency.
@hlxid
Copy link
Member Author

hlxid commented May 23, 2019

Is bootstrap/src/main/resources/dependencies.xml intended to be commited?

@sebinside
Copy link
Member

Great pull request, thank you! dependencies.xml ist not ment to be commited. It would be even better to get the output of sbt dependenciesList directly, but we were not able to do this without external make file.

I will inspect your changes as soon as possible.

@hlxid
Copy link
Member Author

hlxid commented May 24, 2019

I've managed to get the dependencies directly from sbt. See my latest commit.

@sebinside
Copy link
Member

Great work! Tested your code, everything works fine and is well developed. Thank you for your help, we would love to welcome you on the code overflow discord server!

@sebinside sebinside merged commit e58f089 into codeoverflow-org:master May 25, 2019
@hlxid
Copy link
Member Author

hlxid commented May 25, 2019

No problem. Love how Chat Overflow progresses, as Code Overflow 1 was my main impuls to learn coding.

@hlxid hlxid deleted the bootstrap-jcenter-support branch May 25, 2019 13:56
hlxid added a commit to codeoverflow-org/chatoverflow-wiki that referenced this pull request Jul 27, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants