Skip to content

Update NodeJS version image for Foundry VTT - #66

Draft
tracer99 wants to merge 1 commit into
pelican-eggs:mainfrom
tracer99:FoundryVTT-NodeJS-Version-Patch
Draft

Update NodeJS version image for Foundry VTT#66
tracer99 wants to merge 1 commit into
pelican-eggs:mainfrom
tracer99:FoundryVTT-NodeJS-Version-Patch

Conversation

@tracer99

@tracer99 tracer99 commented Jul 6, 2025

Copy link
Copy Markdown

Description

Foundry no longer supports NodeJS 18
The recommend minimum 20.
Using the Generic NodeJS image from https://github.com/pelican-eggs/generic/tree/main/nodejs

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?

I don't believe the following entries under "egg update" apply to my patch as it's a simple image change for an existing egg.

  • 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

Foundry no longer supports NodeJS 18
The recommend minimum 20.
Using the Generic NodeJS image from https://github.com/pelican-eggs/generic/tree/main/nodejs
@rmartinoscar

rmartinoscar commented Jul 7, 2025

Copy link
Copy Markdown
Contributor

Hey could you export the edited egg from your panel and use nodejs LTS if it works instead of EOL please ?

@tracer99

tracer99 commented Jul 7, 2025

Copy link
Copy Markdown
Author

@rmartinoscar
I'm confused. v24 is the current and latest version with planned LTS out passed April 2027. v22 is in LTS maintenance mode only.
https://nodejs.org/en/about/previous-releases

Edit: I think I get the confusion. My note said the minimum version of Node supported by Foundry is 20. But I didn't actually put in the description that I used 24.

@parkervcp

Copy link
Copy Markdown
Contributor

@tracer99 This would have resolved an issue someone had the other day where the fix was to change the image used.

@gOOvER

gOOvER commented Jan 11, 2026

Copy link
Copy Markdown
Contributor

@tracer99 can you fix the conflicts or is this PR deprecated because of draft?

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.

4 participants