Skip to content

build: update dependency sass_api to v18 - #4064

Open
angular-robot wants to merge 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-18-x
Open

angular-robot wants to merge 1 commit into
angular:mainfrom
angular-robot:ng-renovate/sass-api-18-x

Conversation

@angular-robot

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Type Update Change
sass_api dependencies major 17.11.2 → 18.1.0

  • 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 Oct 23, 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_api dependency from version 17.11.2 to 18.1.0 in pubspec.yaml and updates the corresponding lockfile pubspec.lock. The reviewer recommends using a caret version constraint (^18.1.0) instead of pinning the dependency to an exact version to avoid potential version conflicts for consumers.

# Add regular dependencies here.
dependencies:
sass_api: 17.11.2
sass_api: 18.1.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

In Dart, it is standard practice to use caret version constraints (e.g., ^18.1.0) for dependencies in pubspec.yaml rather than pinning to an exact version. Pinning dependencies in a package can lead to strict version conflicts for consumers who might require a different patch or minor version of the same dependency. Using a caret constraint allows pub to resolve compatible minor and patch updates while still ensuring API compatibility under semantic versioning.

  sass_api: ^18.1.0

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.

1 participant