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

Migrated to API8 #15

Open
wants to merge 15 commits into
base: master
Choose a base branch
from
Open

Migrated to API8 #15

wants to merge 15 commits into from

Conversation

masagameplay
Copy link

@masagameplay masagameplay commented Jul 24, 2021

I migrated PacketGate to Sponge API8. I changed in this PR:

  • Switched to Mojang mappings (Sponge's VanillaGradle)
  • Pumped Gradle to 7.0.2
  • Switched Reflections to Mixins
  • Updated README
  • Upgraded build.gradle to Kotlin DSL

I also introduced PacketGateAPI because API8 does not allow providing anything else than their implementations.

@Elikill58
Copy link

For publishing, there is multiple ways:

  • Use github actions
  • Auto-create a pom, to make it usage with jitpack
  • Send jar to repo website that will generate pom file

Personnally, the github actions can be very useful for people. I personnally published the 0.2.0 version on my repo as release. I can setup it if you want, I have some experiences about it.

@masagameplay
Copy link
Author

@Elikill58 feel free to do so. I could not figure out how to do so with CodeMC's docs

* Add workflow

* Use gradle command

* Update build.yml

* Change build directory

* Fix wrong packetSet
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