Skip to content

Commit

Permalink
Improve performance of changelogger-validate-all.sh (#38074)
Browse files Browse the repository at this point in the history
`changelogger version` is unnecessarily calling `git` to fetch the
timestamp and PR number for all the changelog entries, when all it's
really using is the significance. Skip that.

Then, while validating each individual project is reasonably fast, when
there's 117 of them it adds up. We can improve things somewhat by
processing projects in parallel, matching the number of processors in
the system.

On my laptop, the first takes the time from ~16.8s to ~10.7s, and the
second (8 cores) takes it to ~2.3s.

Committed via a GitHub action: https://github.com/Automattic/jetpack/actions/runs/9695953187

Upstream-Ref: Automattic/jetpack@00c1de8
  • Loading branch information
anomiex authored and matticbot committed Jun 27, 2024
1 parent 889b6aa commit 760fa26
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@
"license": "GPL-2.0-or-later",
"require": {},
"require-dev": {
"automattic/jetpack-changelogger": "^4.2.4"
"automattic/jetpack-changelogger": "^4.2.5-alpha"
},
"scripts": {
"test-js": [
Expand Down

0 comments on commit 760fa26

Please sign in to comment.