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

Backend: Add Minecraft Version to jar file name #2941

Merged
merged 4 commits into from
Dec 5, 2024

Conversation

CalMWolfs
Copy link
Collaborator

@CalMWolfs CalMWolfs commented Nov 17, 2024

What

In preperation for the eventual update of skyhanni to supporting more than one minecraft version, I have added the minecraft version to the jar name when it is built and also made the auto updater search specifically for a jar with that version number so that the correct mod file is downloaded.

Now jar names will be in the format of SkyHanni-mc1.8.9-0.28.Beta.13.jar
This change to the updater allows this format to be modified later if desired, as long as the whole mc version (1.8.9) is still somewhere in the jar name.

Changelog Improvements

  • Added the Minecraft version to the mod file name. - CalMWolfs

Changelog Technical Details

  • The auto updater now searches for JAR files that include the correct Minecraft version in their names. - CalMWolfs

@CalMWolfs CalMWolfs added the Soon This Pull Request will be merged within the next couple of betas label Nov 17, 2024
@CalMWolfs CalMWolfs added this to the Version 0.28 milestone Nov 17, 2024
@github-actions github-actions bot added the Backend A backend pull request that will be merged soon label Nov 17, 2024
@hannibal002 hannibal002 merged commit d34ebc0 into hannibal002:beta Dec 5, 2024
6 checks passed
@github-actions github-actions bot removed Soon This Pull Request will be merged within the next couple of betas Backend A backend pull request that will be merged soon labels Dec 5, 2024
@CalMWolfs CalMWolfs deleted the mc-version-in-jar-name branch December 5, 2024 07:07
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