mirror of
https://github.com/community-scripts/ProxmoxVE.git
synced 2026-08-30 12:05:44 +00:00
ceb783d62c
* Post the command that tests a ct/ or install/ change Reviewing a script change meant working out the URL yourself, and the obvious guess is wrong: curling the branch URL alone gives you the ct/ script from the PR and the install/ script from main, because each script pins _CS_DEFAULT_URL to main and that pin is what fills COMMUNITY_SCRIPTS_URL when it is unset. Frequently the install script is the only thing that changed. So the comment spells out both lines, per changed app. Only for scripts already on the core bootstrap. The older one-liner resolves everything from ProxmoxVE/main and ignores the variable, so a command built for it would install main and look like it passed -- worse than no comment. Those are named instead, with what to do about them. pull_request_target for fork PRs, and nothing from the PR is checked out or executed: the file list and the bootstrap line come from the API, and a branch name that is not [A-Za-z0-9._/-]+ stops the run rather than reaching a fenced code block. * Update .github/workflows/pr-test-command.yml Co-authored-by: Sam Heinz <sam@samheinz.com> --------- Co-authored-by: Sam Heinz <sam@samheinz.com>