[PR #4744] [MERGED] Big NodeJS Update: Use Helper Function on all Install-Scripts #5075

Closed
opened 2026-02-26 14:32:59 +03:00 by kerem · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/community-scripts/ProxmoxVE/pull/4744
Author: @MickLesk
Created: 5/27/2025
Status: Merged
Merged: 5/27/2025
Merged by: @michelroegl-brunner

Base: mainHead: nodejs_function


📝 Commits (3)

  • 6f42afe Big NodeJS Update: Use Helper Function on all Install-Scripts
  • 821db3c add jq in node_function
  • 9f9ab83 add more

📊 Changes

58 files changed (+148 additions, -791 deletions)

View changed files

📝 install/adventurelog-install.sh (+3 -17)
📝 install/archivebox-install.sh (+1 -10)
📝 install/bytestash-install.sh (+1 -15)
📝 install/changedetection-install.sh (+1 -10)
📝 install/cronicle-install.sh (+1 -14)
📝 install/cross-seed-install.sh (+1 -14)
📝 install/cryptpad-install.sh (+1 -11)
📝 install/dashy-install.sh (+1 -14)
📝 install/docmost-install.sh (+3 -14)
📝 install/elementsynapse-install.sh (+2 -13)
📝 install/excalidraw-install.sh (+1 -12)
📝 install/flowiseai-install.sh (+1 -11)
📝 install/frigate-install.sh (+1 -7)
📝 install/ghost-install.sh (+2 -12)
📝 install/gomft-install.sh (+3 -22)
📝 install/grist-install.sh (+1 -9)
📝 install/habitica-install.sh (+4 -14)
📝 install/homepage-install.sh (+1 -11)
📝 install/iobroker-install.sh (+1 -10)
📝 install/karakeep-install.sh (+1 -7)

...and 38 more files

📄 Description

✍️ Description

  • remove all nodejs parts in install scripts
  • use the tools.func for nodejs installs
  • remove gpg / gnupg deps

add jq in install_nodes_and_modules function (for crawling special pnpm, yarn versions)

Need to test after merge:

  1. NodeBB
  2. PingvinShare
  3. ps5mqtt
  4. wikijs

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.

🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/community-scripts/ProxmoxVE/pull/4744 **Author:** [@MickLesk](https://github.com/MickLesk) **Created:** 5/27/2025 **Status:** ✅ Merged **Merged:** 5/27/2025 **Merged by:** [@michelroegl-brunner](https://github.com/michelroegl-brunner) **Base:** `main` ← **Head:** `nodejs_function` --- ### 📝 Commits (3) - [`6f42afe`](https://github.com/community-scripts/ProxmoxVE/commit/6f42afe22c2900790b5675ab5e9df88b0d72381e) Big NodeJS Update: Use Helper Function on all Install-Scripts - [`821db3c`](https://github.com/community-scripts/ProxmoxVE/commit/821db3c07976b19703200a3a8dd84d669db34665) add jq in node_function - [`9f9ab83`](https://github.com/community-scripts/ProxmoxVE/commit/9f9ab83b5087e0e295ff1891e362316e1ce9750b) add more ### 📊 Changes **58 files changed** (+148 additions, -791 deletions) <details> <summary>View changed files</summary> 📝 `install/adventurelog-install.sh` (+3 -17) 📝 `install/archivebox-install.sh` (+1 -10) 📝 `install/bytestash-install.sh` (+1 -15) 📝 `install/changedetection-install.sh` (+1 -10) 📝 `install/cronicle-install.sh` (+1 -14) 📝 `install/cross-seed-install.sh` (+1 -14) 📝 `install/cryptpad-install.sh` (+1 -11) 📝 `install/dashy-install.sh` (+1 -14) 📝 `install/docmost-install.sh` (+3 -14) 📝 `install/elementsynapse-install.sh` (+2 -13) 📝 `install/excalidraw-install.sh` (+1 -12) 📝 `install/flowiseai-install.sh` (+1 -11) 📝 `install/frigate-install.sh` (+1 -7) 📝 `install/ghost-install.sh` (+2 -12) 📝 `install/gomft-install.sh` (+3 -22) 📝 `install/grist-install.sh` (+1 -9) 📝 `install/habitica-install.sh` (+4 -14) 📝 `install/homepage-install.sh` (+1 -11) 📝 `install/iobroker-install.sh` (+1 -10) 📝 `install/karakeep-install.sh` (+1 -7) _...and 38 more files_ </details> ### 📄 Description ## ✍️ Description - remove all nodejs parts in install scripts - use the tools.func for nodejs installs - remove gpg / gnupg deps add jq in install_nodes_and_modules function (for crawling special pnpm, yarn versions) Need to test after merge: 1. NodeBB 2. PingvinShare 3. ps5mqtt 4. wikijs ## ✅ Prerequisites (**X** in brackets) - [x] **Self-review completed** – Code follows project standards. - [ ] **Tested thoroughly** – Changes work as expected. - [x] **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. - [x] ✨ **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. - [x] 🔧 **Refactoring / Code Cleanup** – Improves readability or maintainability without changing functionality. - [ ] 📝 **Documentation update** – Changes to `README`, `AppName.md`, `CONTRIBUTING.md`, or other docs. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
kerem 2026-02-26 14:32:59 +03:00
Sign in to join this conversation.
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
starred/ProxmoxVE#5075
No description provided.