Skip to content

Conversation

alan-agius4
Copy link
Contributor

Within our module.bazel file when describing the version of typescript to use for rules_ts, we use ts_version instead of ts_version_from to prevent our package.json file from being part of the set of files used to calculate the sha for the lock file. Any unrelated change to the version of the typescript file would end up causing our lockfile to be out of date. This amount of churn has proven to be too much for our current setup. We instead now test to validate the versions defined in the package.json and MODULE.bazel files match.

Backport of https://github.com/angular/angular/pull/63431/commits

Within our module.bazel file when describing the version of typescript to use for rules_ts, we use ts_version instead of ts_version_from to prevent our package.json file from being
part of the set of files used to calculate the sha for the lock file.  Any unrelated change to the version of the typescript file would end up causing our lockfile to be out of date.
This amount of churn has proven to be too much for our current setup. We instead now test to validate the versions defined in the package.json and MODULE.bazel files match.
@alan-agius4 alan-agius4 requested a review from crisbeto August 28, 2025 07:08
@alan-agius4 alan-agius4 requested a review from a team as a code owner August 28, 2025 07:08
@alan-agius4 alan-agius4 added the target: minor This PR is targeted for the next minor release label Aug 28, 2025
@alan-agius4 alan-agius4 requested a review from a team as a code owner August 28, 2025 07:08
@alan-agius4 alan-agius4 requested review from andrewseguin and removed request for a team August 28, 2025 07:08
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Aug 28, 2025
@alan-agius4 alan-agius4 removed request for a team and andrewseguin August 28, 2025 07:09
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Aug 28, 2025
@crisbeto crisbeto merged commit 9081e23 into angular:main Aug 28, 2025
29 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants