Skip to content

Release: Rewrite SVN publication scripts in Node.js #84034

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
ciampo merged 3 commits into from
Oct 6, 2026
Merged

Release: Rewrite SVN publication scripts in Node.js #84034

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
71febbc
Release: Rewrite SVN publication scripts in Node.js
ciampo Oct 2, 2026
9cdf55f
Release: Fix SVN publisher path verification
ciampo Oct 2, 2026
d04c6ea
Release: Stream SVN publication output
ciampo Oct 5, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 6 additions & 3 deletions .github/workflows/unit-test.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -83,11 +83,14 @@ jobs:
- name: Install Subversion
run: sudo apt-get update -y && sudo apt-get install -y subversion

- name: Lint release scripts
run: shellcheck tools/release/*.sh tools/release/test/*.sh
- name: Use desired version of Node.js
uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
with:
node-version-file: .nvmrc
package-manager-cache: false

- name: Test SVN publication and recovery
run: bash tools/release/test/publish-to-svn.sh
run: node --test tools/release/test-publish-to-svn.mts

unit-js:
name: JavaScript (Node.js ${{ matrix.node }}) ${{ matrix.shard }}
Expand Down
16 changes: 14 additions & 2 deletions .github/workflows/upload-release-to-plugin-repo.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -284,6 +284,12 @@ jobs:
sparse-checkout: tools/release
persist-credentials: false

- name: Use desired version of Node.js
uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
with:
node-version-file: .nvmrc
package-manager-cache: false

- name: Install Subversion
run: |
sudo apt-get update -y && sudo apt-get install -y subversion
Expand Down Expand Up @@ -318,7 +324,7 @@ jobs:
path: trunk

- name: Publish and verify the release
run: bash tools/release/publish-to-svn.sh "$GITHUB_WORKSPACE/trunk" trunk
run: node tools/release/publish-to-svn.mts "$GITHUB_WORKSPACE/trunk" trunk

upload-tag:
name: Publish as tag
Expand Down Expand Up @@ -349,6 +355,12 @@ jobs:
sparse-checkout: tools/release
persist-credentials: false

- name: Use desired version of Node.js
uses: actions/setup-node@820762786026740c76f36085b0efc47a31fe5020 # v7.0.0
with:
node-version-file: .nvmrc
package-manager-cache: false

- name: Install Subversion
run: |
sudo apt-get update -y && sudo apt-get install -y subversion
Expand All @@ -375,4 +387,4 @@ jobs:
path: ${{ needs.get-release-metadata.outputs.version }}

- name: Publish and verify the release
run: bash tools/release/publish-to-svn.sh "$GITHUB_WORKSPACE/$VERSION" tag
run: node tools/release/publish-to-svn.mts "$GITHUB_WORKSPACE/$VERSION" tag
4 changes: 3 additions & 1 deletion tools/release/package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -47,6 +47,8 @@
"build-plugin-zip": "node ./build-plugin-zip.mts",
"cherry-pick": "node ./cherry-pick.mjs",
"generate-php-sync-issue": "node ./generate-php-sync-issue.mjs",
"log-performance-results": "node ./log-performance-results.js"
"log-performance-results": "node ./log-performance-results.js",
"publish-to-svn": "node ./publish-to-svn.mts",
"test:svn": "node --test ./test-publish-to-svn.mts"
}
}
Loading
Loading