mirror of
https://github.com/community-scripts/ProxmoxVE.git
synced 2026-03-03 16:15:54 +00:00
github: improvements for node drift wf (#12309)
* fix(workflow): improve Node.js version drift detection accuracy 1. Fix source URL regex: now captures 'Github: https://github.com/...' pattern (pipe-separated), not just '# Source: https://github.com/...' This was causing ~50 scripts to show 'No GitHub source' 2. Fix semver comparison: engines.node constraints like '>=18.0.0' no longer flag version 22 as drift. >= and ^ constraints are now properly evaluated (our_version >= min_major = satisfied) 3. Add fallback detection: when no Dockerfile or engines.node is found, check .nvmrc and .node-version files for Node version hints 4. Add subdirectory search: Dockerfile and package.json are now found via GitHub API tree search, not just in repo root 5. Use GitHub API to detect default branch instead of guessing main/master/dev with multiple HEAD requests * fix typos in node_version * runs on vps
This commit is contained in:
committed by
GitHub
parent
6df91f54af
commit
dd4a15fff1
@@ -39,7 +39,7 @@ $STD apt install -y \
|
||||
texlive-xetex
|
||||
msg_ok "Installed Dependencies"
|
||||
|
||||
NODE_VERSION=22 NODE_MODULE="bun" setup_nodejs
|
||||
NODE_VERSION="22" NODE_MODULE="bun" setup_nodejs
|
||||
fetch_and_deploy_gh_release "ConvertX" "C4illin/ConvertX" "tarball" "latest" "/opt/convertx"
|
||||
|
||||
msg_info "Installing ConvertX"
|
||||
|
||||
@@ -21,7 +21,7 @@ msg_ok "Installed Dependencies"
|
||||
|
||||
PG_VERSION="17" setup_postgresql
|
||||
PG_DB_NAME="joplin" PG_DB_USER="joplin" setup_postgresql_db
|
||||
NODE_VERSION=24 NODE_MODULE="yarn,npm,pm2" setup_nodejs
|
||||
NODE_VERSION="24" NODE_MODULE="yarn,npm,pm2" setup_nodejs
|
||||
mkdir -p /opt/pm2
|
||||
export PM2_HOME=/opt/pm2
|
||||
$STD pm2 install pm2-logrotate
|
||||
|
||||
@@ -21,7 +21,7 @@ $STD apt install -y \
|
||||
expect
|
||||
msg_ok "Dependencies installed."
|
||||
|
||||
NODE_VERSION=24 setup_nodejs
|
||||
NODE_VERSION="24" setup_nodejs
|
||||
fetch_and_deploy_gh_release "ProxmoxVE-Local" "community-scripts/ProxmoxVE-Local" "tarball"
|
||||
|
||||
msg_info "Installing PVE Scripts local"
|
||||
|
||||
Reference in New Issue
Block a user