Skip to content

build: lock file maintenance#3558

Merged
alan-agius4 merged 2 commits intoangular:mainfrom
alan-agius4:pr-takeover-3550
Mar 17, 2026
Merged

build: lock file maintenance#3558
alan-agius4 merged 2 commits intoangular:mainfrom
alan-agius4:pr-takeover-3550

Conversation

@alan-agius4
Copy link
Copy Markdown
Contributor

See associated pull request for more information.

Closes #3550 as a pr takeover

angular-robot and others added 2 commits March 17, 2026 16:37
See associated pull request for more information.

Closes angular#3550 as a pr takeover
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Mar 17, 2026
@angular-robot angular-robot bot added the area: build & ci Related the build and CI infrastructure of the project label Mar 17, 2026
Copy link
Copy Markdown
Member

@josephperrott josephperrott left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates dependency lock files. While most changes are routine version bumps, the update to bazel/rules/rules_sass/pubspec.lock includes an increase in the minimum required Dart SDK version, which could be a breaking change. Please review the specific comment for details.

version: "3.1.3"
sdks:
dart: ">=3.7.2 <4.0.0"
dart: ">=3.9.0 <4.0.0"
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

medium

The minimum Dart SDK constraint has been raised from >=3.7.2 to >=3.9.0. This is a potentially breaking change for developers using older Dart SDK versions (between 3.7.2 and 3.9.0). If this project needs to maintain compatibility with older SDKs, this change should be reverted, which may require pinning some dependencies to older versions.

@alan-agius4 alan-agius4 merged commit 9de194d into angular:main Mar 17, 2026
13 checks passed
@alan-agius4
Copy link
Copy Markdown
Contributor Author

This PR was merged into the repository. The changes were merged into the following branches:

@alan-agius4 alan-agius4 deleted the pr-takeover-3550 branch March 17, 2026 16:53
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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants