Skip to content

Conversation

@SainathPoojary
Copy link
Contributor

What?

Closes #69684

PR adds missing dependencies for the @wordpress/stylelint-scss package.

Testing Instructions

Verify that the change makes sense.

@SainathPoojary SainathPoojary requested a review from ntwb as a code owner March 24, 2025 19:00
@github-actions
Copy link

github-actions bot commented Mar 24, 2025

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: SainathPoojary <[email protected]>
Co-authored-by: Mamaduka <[email protected]>
Co-authored-by: anomiex <[email protected]>

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

@t-hamano t-hamano added [Type] Code Quality Issues or PRs that relate to code quality [Tool] stylelint config /packages/stylelint-config labels Mar 26, 2025
"peerDependencies": {
"stylelint": "^16.8.2"
"stylelint": "^16.8.2",
"stylelint-scss": "^6.4.0"
Copy link
Member

Choose a reason for hiding this comment

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

@SainathPoojary, how did you choose this version?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hey @Mamaduka,

I chose ^6.4.0 for stylelint-scss based on the version already used in the package-lock.json, where it’s pulled in as a dependency of stylelint-config-recommended-scss.

It was using "stylelint-scss": "^6.4.0" as it's dependency

"packages/stylelint-config/node_modules/stylelint-config-recommended-scss": {
	"version": "14.1.0",
	"resolved": "https://registry.npmjs.org/stylelint-config-recommended-scss/-/stylelint-config-recommended-scss-14.1.0.tgz",
	"integrity": "sha512-bhaMhh1u5dQqSsf6ri2GVWWQW5iUjBYgcHkh7SgDDn92ijoItC/cfO/W+fpXshgTQWhwFkP1rVcewcv4jaftRg==",
	"dependencies": {
		"postcss-scss": "^4.0.9",
		"stylelint-config-recommended": "^14.0.1",
		"stylelint-scss": "^6.4.0"
	},
	"engines": {
		"node": ">=18.12.0"
	},
	"peerDependencies": {
		"postcss": "^8.3.3",
		"stylelint": "^16.6.1"
	},
	"peerDependenciesMeta": {
		"postcss": {
			"optional": true
		}
	}
},

Copy link
Member

@Mamaduka Mamaduka Apr 11, 2025

Choose a reason for hiding this comment

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

Thanks for the clarifications, @SainathPoojary!

Copy link
Member

@Mamaduka Mamaduka left a comment

Choose a reason for hiding this comment

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

Thanks, @SainathPoojary!

@Mamaduka Mamaduka added the props-bot Manually triggers Props Bot to ensure the list of props is up to date. label Apr 11, 2025
@github-actions github-actions bot removed the props-bot Manually triggers Props Bot to ensure the list of props is up to date. label Apr 11, 2025
@Mamaduka Mamaduka merged commit 59a85a6 into WordPress:trunk Apr 11, 2025
71 checks passed
@github-actions github-actions bot added this to the Gutenberg 20.7 milestone Apr 11, 2025
chriszarate pushed a commit to chriszarate/gutenberg that referenced this pull request Jul 1, 2025
…69685)

* Stylelint-config: Add stylelint-scss as a peer dependency
* Stylelint-config: Update package-lock.json

Co-authored-by: SainathPoojary <[email protected]>
Co-authored-by: Mamaduka <[email protected]>
Co-authored-by: anomiex <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Tool] stylelint config /packages/stylelint-config [Type] Code Quality Issues or PRs that relate to code quality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

@wordpress/stylelint-config is missing a dependency or peer dependency on stylelint-scss

3 participants