chore(deps-dev): bump stylelint-config-sass-guidelines from 11.1.0 to 13.0.0#5873
Conversation
Bumps [stylelint-config-sass-guidelines](https://github.com/bjankord/stylelint-config-sass-guidelines) from 11.1.0 to 13.0.0. - [Release notes](https://github.com/bjankord/stylelint-config-sass-guidelines/releases) - [Changelog](https://github.com/bjankord/stylelint-config-sass-guidelines/blob/main/CHANGELOG.md) - [Commits](bjankord/stylelint-config-sass-guidelines@v11.1.0...v13.0.0) --- updated-dependencies: - dependency-name: stylelint-config-sass-guidelines dependency-version: 13.0.0 dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
dependabot
Bot
added
dependencies
rtibblesbot
left a comment
rtibblesbot
left a comment
There was a problem hiding this comment.
Dependency Update Review
Package: stylelint-config-sass-guidelines 11.1.0 → 13.0.0 (multi-major via v12.0.0, v12.1.0, v13.0.0)
Semver risk: Major
Dependency type: devDependency (linting config)
CI status: Passing
Changelog Analysis
Sources consulted: PR body release notes and changelog for v12.0.0, v12.1.0, v13.0.0.
Notable changes:
- v12.0.0: Restored stylistic rules via
@stylistic/stylelint-plugin(removed in v10); adds rules for braces, hex case, indentation, quotes, etc. - v12.1.0: Transitive dep bumps only, no rule changes
- v13.0.0: Config rule change —
@stylistic/function-parentheses-space-inside: "never"→"never-single-line"(Prettier compatibility); min Node bumped to 20.19.0;stylelint-scssbumped 6.6.0 → 7.0.0;@stylistic/stylelint-pluginbumped 3.0.1 → 5.0.1
Breaking changes: The v12 stylistic rule additions would have surfaced as lint failures — CI passes, so no code violations were introduced.
Security fixes: None.
Compatibility Assessment
- Node: v13.0.0 requires
>=20.19.0; project runs20.19.3✓ - Code changes required: No — CI confirms no rule violations from the new/changed rules
- Peer dependency mismatch:
stylelint-config-sass-guidelines@13.0.0declarespeerDependencies: { stylelint: "^17.1.0" }. Its transitive deps also escalate requirements:@stylistic/stylelint-plugin@5.1.0requiresstylelint@^17.6.0, andstylelint-scss@7.0.0requiresstylelint@^16.8.2 || ^17.0.0. The project pinsstylelint@^15.11.0. pnpm resolves with a peer mismatch (visible in the lockfile snapshot key13.0.0(…)(stylelint@15.11.0(…))). CI passes — the config is practically compatible with stylelint 15 — but the project is running outside the declared support envelope. See inline comment.
Recommendation
Functionally safe to merge — dev dependency, CI passing, no code changes required. A follow-up to upgrade stylelint from v15 → v17 would close the peer dependency gap.
@rtibblesbot's comments are generated by an LLM, and should be evaluated accordingly
How was this generated?
Reviewed the pull request diff checking for:
- Correctness: bugs, edge cases, undocumented behavior, resource leaks, hardcoded values
- Design: unnecessary complexity, naming, readability, comment accuracy, redundant state
- Architecture: duplicated concerns, minimal interfaces, composition over inheritance
- Testing: behavior-based assertions, mocks only at hard boundaries, accurate coverage
- Completeness: missing dependencies, unupdated usages, i18n, accessibility, security
- Principles: DRY (same reason to change), SRP, Rule of Three (no premature abstraction)
- Checked CI status and linked issue acceptance criteria
- For UI changes: inspected screenshots for layout, visual completeness, and consistency
| "stylelint-config-recess-order": "4.6.0", | ||
| "stylelint-config-recommended-scss": "14.1.0", | ||
| "stylelint-config-sass-guidelines": "11.1.0", | ||
| "stylelint-config-sass-guidelines": "13.0.0", |
There was a problem hiding this comment.
suggestion: stylelint-config-sass-guidelines@13.0.0 requires stylelint@^17.1.0 as a peer, and its transitive deps (@stylistic/stylelint-plugin@5.1.0 → ^17.6.0, stylelint-scss@7.0.0 → ^16.8.2 || ^17.0.0) also require stylelint 16+. The project currently pins stylelint@^15.11.0. pnpm resolves the mismatch silently and CI passes, so the linting works in practice — but consider a follow-up PR to upgrade stylelint to v17 to stay within the supported peer range.
1 participant
Bumps stylelint-config-sass-guidelines from 11.1.0 to 13.0.0.
Release notes
Sourced from stylelint-config-sass-guidelines's releases.
Changelog
Sourced from stylelint-config-sass-guidelines's changelog.
Commits
873156eReleased version 13.0.0d9f8a3aUpdate CHANGELOG.md3140138Update package-lock (#373)1a7ffa2Update dependencies (#372)f04c797Update test expectations (#369)ea7f5bcReleased version 12.1.0f99bfe8Update CHANGELOG.md0f1b73fBump stylelint-scss from 6.5.1 to 6.6.0 (#356)45dc7bbBump postcss from 8.4.44 to 8.4.45 (#355)3fb8e3eBump@stylistic/stylelint-pluginfrom 2.1.2 to 3.0.1 (#350)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore this major versionwill close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this minor versionwill close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)@dependabot ignore this dependencywill close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)