Skip to content

Update NeoForge egg reguler expression to include suffexes like "-beta"#116

Open
AdamBougher wants to merge 2 commits intopelican-eggs:mainfrom
AdamBougher:patch-1
Open

Update NeoForge egg reguler expression to include suffexes like "-beta"#116
AdamBougher wants to merge 2 commits intopelican-eggs:mainfrom
AdamBougher:patch-1

Conversation

@AdamBougher
Copy link
Copy Markdown

@AdamBougher AdamBougher commented Feb 18, 2026

Description

When trying to start a Neoforge server, it wasn't letting me set the current Neoforge version, i found it was an issue with the regex not allowing any letters, i have added that ability to the check

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

@AdamBougher AdamBougher marked this pull request as draft February 18, 2026 04:14
@AdamBougher AdamBougher marked this pull request as ready for review February 18, 2026 04:29
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.

1 participant