Skip to content
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

build: shadow jar - all deps in one jar #244

Merged
merged 3 commits into from Jan 4, 2021

Conversation

stachu540
Copy link
Contributor

Prerequisites for Code Changes

  • This pull request follows the code style of the project
  • I have tested this feature

Changes Proposed

  1. Apply plugin com.github.johnrengelman.shadow - Closes FatJar in Maven repo #226

Additional Information

In #226 where providing this plugin and applying shaded word into archiveClassifier for shadowJar task, will automatically adds into MavenPublication.

All those changes are be tested before the pull request.

jar with "shaded" classifier
Copy link
Member

@PhilippHeuer PhilippHeuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should we relocate all packages to avoid conflicts with other dependencies and also with ourselves? (ie. to allow ppl to use the shaded TwitchChat 1.0 together with shaded TwitchPubSub 1.1 as single modules)

https://imperceptiblethoughts.com/shadow/configuration/relocation/#automatically-relocating-dependencies

We would just need to use a unique prefix for each module.

deployment.gradle Outdated Show resolved Hide resolved
PhilippHeuer and others added 2 commits January 4, 2021 19:31
Co-authored-by: Damian Staszewski <359222+stachu540@users.noreply.github.com>
Copy link
Member

@PhilippHeuer PhilippHeuer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had to change it a little because the example in the shadow documentation wasn't working, but it works now.

@PhilippHeuer PhilippHeuer merged commit 8bfeadd into twitch4j:develop Jan 4, 2021
@stachu540
Copy link
Contributor Author

#226 Closed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants