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

add official 1.20.6 support #10

Closed
wants to merge 7 commits into from

Conversation

NonSwag
Copy link

@NonSwag NonSwag commented Jun 6, 2024

Description

This PR adds official support for 1.20.6

  • cloud command framework was updated to utilize the latest addition of paper brigadier
  • use gooolers shadowJar fork for java 21 support
  • updated hangar publish and removed dead code from publication task
  • updated workflows to use java 21
  • removed dead versions from "supported" list

Checklist

  • I included all information required in the sections above
  • I tested my changes and approved their functionality
  • I ensured my changes do not break other parts of the code

@NonSwag NonSwag requested a review from TheMeinerLP as a code owner June 6, 2024 15:16
build.gradle.kts Show resolved Hide resolved
gradle/wrapper/gradle-wrapper.properties Outdated Show resolved Hide resolved
build.gradle.kts Show resolved Hide resolved
build.gradle.kts Show resolved Hide resolved
@TheMeinerLP
Copy link
Collaborator

It would be nice to have changes that are not specific to PR. “Dependency updates, Gradle Update” should be made separately so that not everything has to be tested at once. If necessary, a component can already break something else

@NonSwag
Copy link
Author

NonSwag commented Jun 6, 2024

I could split it up into plugins, java and gradle update
and cloud update

@TheMeinerLP
Copy link
Collaborator

That would be perfect, I would be very happy with that

@NonSwag
Copy link
Author

NonSwag commented Jun 7, 2024

split up into #11, #12 and #13

@NonSwag NonSwag closed this Jun 7, 2024
@NonSwag NonSwag deleted the mc-1.20.6 branch June 7, 2024 16:35
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