Skip to content
This repository has been archived by the owner on May 14, 2024. It is now read-only.

Updated spigodt install script to suport java 21 #2919

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

Conversation

Pomdre
Copy link

@Pomdre Pomdre commented Apr 23, 2024

Description

Update spigodt install script to use java 21 from openjdk when building spigodt

Checklist for all submissions

  • Have you followed the guidelines in our Contributing document?
  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you tested and reviewed your changes with confidence that everything works?
  • Did you branch your changes and PR from that branch and not from your master branch?
    • If not, why?:
  • You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  • The egg was exported from the panel

New egg Submissions

  1. Does your submission pass tests (server is connectable)?
  2. Does your egg use a custom docker image?
    • Have you tried to use a generic image?
    • Did you PR the necessary changes to make it work?
  3. Have you added the egg to the main README.md and any other README files in subdirectories of the egg (e.g /game_eggs) according to the alphabetical order?
  4. Have you added a unique README.md for the egg you are adding according to the alphabetical order?
  5. You verify that the start command applied does not use a shell script
    • If some script is needed then it is part of a current yolk or a PR to add one
  6. The egg was exported from the panel

@QuintenQVD0
Copy link
Collaborator

This repo is not used anymore, any PR Should be send to their sub repo for you https://github.com/pelican-eggs/minecraft

and you did not fully export the egg from the panel, make all changes in a panel, then export and fully replace the file here.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants