Skip to content

Conversation

@tremor021
Copy link
Member

✍️ Description

  • Bumped Node.js to 24

🔗 Related PR / Issue

Link: #9401

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

@tremor021 tremor021 self-assigned this Nov 24, 2025
@tremor021 tremor021 requested a review from a team as a code owner November 24, 2025 14:42
@github-actions github-actions bot added bugfix update script A change that updates a script labels Nov 24, 2025
@tremor021 tremor021 linked an issue Nov 24, 2025 that may be closed by this pull request
2 tasks
@MickLesk
Copy link
Member

in update_script too?

Set NODE_VERSION and NODE_MODULE for Node.js setup.
@michelroegl-brunner michelroegl-brunner merged commit b4bce49 into main Nov 24, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix update script A change that updates a script

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Install fails for Joplin-Server

4 participants