Skip to content

build: update dependency sass_api to v17.7.7#3870

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-17-x
Jul 24, 2026
Merged

build: update dependency sass_api to v17.7.7#3870
alan-agius4 merged 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-17-x

Conversation

@angular-robot

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
sass_api dependencies minor 17.6.017.7.7

  • If you want to rebase/retry this PR, check this box

See associated pull request for more information.
@angular-robot angular-robot added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only] labels Jul 24, 2026

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

Copy link
Copy Markdown

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 the sass and sass_api dependencies to versions 1.101.7 and 17.7.7 in pubspec.lock and pubspec.yaml. The reviewer suggested wrapping the version number in quotes in pubspec.yaml to ensure consistent and safe YAML parsing.

# Add regular dependencies here.
dependencies:
sass_api: 17.6.0
sass_api: 17.7.7

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

It is a recommended best practice to wrap version numbers in quotes in pubspec.yaml. While 17.7.7 is parsed as a string due to multiple dots, simpler version numbers (like 17 or 17.10) can be incorrectly parsed as integers or floats by YAML parsers, leading to unexpected behavior. Quoting all version numbers ensures consistent and safe parsing.

  sass_api: "17.7.7"

@alan-agius4
alan-agius4 merged commit fa0a1a7 into angular:main Jul 24, 2026
17 checks passed
@alan-agius4

Copy link
Copy Markdown
Contributor

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

@angular-robot
angular-robot deleted the ng-renovate/sass-api-17-x branch July 24, 2026 10:02
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: automation This PR is targeted to only merge into the branch defined in Github [bot use only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants