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

Migrate to Gradle #36

Merged
merged 6 commits into from
Jan 7, 2025
Merged

Migrate to Gradle #36

merged 6 commits into from
Jan 7, 2025

Conversation

calcastor
Copy link
Contributor

@calcastor calcastor commented Dec 28, 2024

This migrates Events to using gradle in place of maven.

The new jar is smaller due to some unnecessary maven files being left out as well as simply using PGM's shaded jdom2. It has been tested to the extent that a PUG on OCC launches and cycles to the map set by the user.

Draft because I haven't tested it in practice, but reviews are fine. I will probably also simplify the build structure in comparison to PGMDev/PGM#1429 and PGMDev/Community#77, from which this PR heavily borrows, since Events is unlikely to need platform implementations to work on both 1.8 and modern.

This PR also sets up maven github package publishing.

Later: migrate to Adventure, since bungeecord-chat is deprecated and will likely be removed in the future particularly following Paper hard fork; publish to repo.pgm.fyi automatically.

Supersedes #35

@calcastor calcastor force-pushed the gradle branch 2 times, most recently from 2209604 to 679e045 Compare December 29, 2024 05:59
Signed-off-by: BT (calcastor/mame) <[email protected]>
@calcastor calcastor marked this pull request as ready for review December 29, 2024 07:09
Signed-off-by: BT (calcastor/mame) <[email protected]>
Signed-off-by: BT (calcastor/mame) <[email protected]>
Signed-off-by: BT (calcastor/mame) <[email protected]>
@Pablete1234 Pablete1234 merged commit f9691d0 into PGMDev:master Jan 7, 2025
2 checks passed
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