moditect / layrry

A Runner and API for Layered Java Applications
Apache License 2.0
332 stars 33 forks source link

Configure binary distribution #48

Closed aalmiray closed 3 years ago

aalmiray commented 3 years ago

This PR is the first step towards publication via SDKMAN! (#27)

gunnarmorling commented 3 years ago

Looks good. How about having a tar.gz too?

aalmiray commented 3 years ago

Sure, why not 😄

aalmiray commented 3 years ago

What do you think of renaming the shaded artifact? I looked for options to keep the status quo but either the binary contains the wrong JAR or the incorrect shaded artifact (-shaded classifier) is published to Maven Central.

If accepted this changes requires updating the README and the jbang-catalog.

gunnarmorling commented 3 years ago

What do you think of renaming the shaded artifact?

I don't have any strong preference really. Whatever works :)

aalmiray commented 3 years ago

OK, then let's keep it as it is in this PR as that requires less conf in Maven.

gunnarmorling commented 3 years ago

+1

aalmiray commented 3 years ago

@gunnarmorling please merge whenever you're ready