Skip to content

Allow entity-particles to be built on JitPack#2

Merged
randombyte-developer merged 1 commit intorandombyte-developer:masterfrom
Aaron1011:fix/jitpack
Jul 16, 2018
Merged

Allow entity-particles to be built on JitPack#2
randombyte-developer merged 1 commit intorandombyte-developer:masterfrom
Aaron1011:fix/jitpack

Conversation

@Aaron1011
Copy link
Contributor

This commits makes two changes:

  • Only enable the prompsign plugin when not on JitPack
  • Make the maven 'install' task depedent on shadowJar, to ensure that
    shadowing happens properly when JitPack runs './gradlew install'

This commits makes two changes:
* Only enable the prompsign plugin when not on JitPack
* Make the maven 'install' task depedent on shadowJar, to ensure that
shadowing happens properly when JitPack runs './gradlew install'
@randombyte-developer
Copy link
Owner

The same as with ByteItems, though I think based on #1 that you are trying to include this plugin somehow in a mod plugin pack, right?

@Aaron1011
Copy link
Contributor Author

I'm adding a integration with PixelmonPlaceholders. At dev time, having a shaded jar available allows me to easily run PixelmonPlaceholders from my IDE. Otherwise, I need to temporarily remove the dependency from Intellij's settings and download entity-particles separately, due to the missing bStats jar.

@randombyte-developer
Copy link
Owner

Ok

@randombyte-developer randombyte-developer merged commit fe3053a into randombyte-developer:master Jul 16, 2018
randombyte-developer added a commit that referenced this pull request Jul 16, 2018
- Added tracking of entities with particles to optimize timings by a lot
- Breaking change: The plugin classes in the final fat jar aren't relocated anymore #3
- Added support for running the plugin on the client #1
- Now also relocating everything when built by JitPack #2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments