-
Notifications
You must be signed in to change notification settings - Fork 4.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Bump Stylelint dependencies to latest v14.x compatible releases #51192
Conversation
Size Change: +2.68 kB (0%) Total Size: 1.4 MB
ℹ️ View Unchanged
|
stylelint-config-recommended
& stylelint-config-recommended-scss
to v12
Flaky tests detected in 9605df5. 🔍 Workflow run URL: https://github.com/WordPress/gutenberg/actions/runs/5154116050
|
There is now another more up-to-date PR from @mikeybinns that is fully functional, ready for review and testing: Thank you so much for your contribution! |
What?
Bumps
stylelint
from v14.2 to v14.16.1Bumps
stylelint-config-recommended
from v6.0.0 to v9.0.0Bumps
stylelint-config-recommended-scss
from v5.0.2 to v8.0.0Dependencies changes remove
^
semver caret, because Lerna is not a fan of installing specific versions, it's moot as each of these dependency bumps are the latest andlast releases in each of these major versions of each packageWhy?
With the Gutenberg project blocked from using Stylelint v15 because of the removal of "stylistic" rules this PR seeks to update Stylelint dependencies to latest compatible with Stylelint v14
How?
Packages were updated using the guide here: https://github.com/WordPress/gutenberg/blob/trunk/packages/README.md
Each package was first removed from each package from the
dependencies
section of the packagepackage.json
fileEach package upgrade was then added using Lerna v5.x
lerna add stylelint-config-recommended@9 packages/stylelint-config/
lerna add stylelint-config-recommended-scss@8 packages/stylelint-config/
lerna add [email protected] packages/scripts/
Testing Instructions
bump-stylelint-config-recommended-packages
branchnpm install
npm run build
npm run lint:css
Testing Instructions for Keyboard
Screenshots or screencast
Notes
See this issue comment in relation to why Stylelint is not being bumped to v15 at this stage #50336 (comment)
CI Failures
TypeError [ERR_INVALID_URL_SCHEME]: The URL must be of scheme file
issue