tools: Improve speed of tools/changelogger-release.sh
#39146
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Proposed changes:
The script has gotten a bit slow, since it was running
check-intra-monorepo-deps.sh
individually for each project to pass either-u
or-U
depending on whether the project was included in the release or not.We can speed it up by running
check-intra-monorepo-deps.sh -U
just once, and then looking at the git diff to see whether any changelog entries seem to be needed.Also, we can then drop the
PACKAGE_VERSIONS_CACHE
thing fromcheck-intra-monorepo-deps.sh
that was done in a prior speedup attempt.And finally, skip trying to handle the "monorepo" pseudo-package, it won't work and prevents releases of projects depended on by the monorepo root.
Other information:
Jetpack product discussion
None
Does this pull request change what data or activity we track or use?
No
Testing instructions: