Skip to content
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

Infrastructure: Move to Stylelint 15 #2613

Merged
merged 1 commit into from
Feb 13, 2023
Merged

Conversation

nschonni
Copy link
Contributor

@nschonni nschonni commented Feb 13, 2023

https://stylelint.io/migration-guide/to-15/ recommends migrating to Prettier for all the formatting parts directly. This updates the related hooks and dependencies to implement that approach, along with fixing some new minor violations


WAI Preview Link (Last built on Mon, 13 Feb 2023 21:44:09 GMT).

Copy link
Contributor

@howard-e howard-e left a comment

Choose a reason for hiding this comment

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

Whoa 76 rules deprecated!

Looks good to me

@@ -56,15 +56,14 @@
"node-fetch": "^2.6.7",
"prettier": "^2.7.1",
"selenium-webdriver": "^4.3.0",
"stylelint": "^14.5.3",
Copy link
Contributor

Choose a reason for hiding this comment

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

Happy to see this get cleaned up a bit.

@howard-e howard-e merged commit d95c6a4 into w3c:main Feb 13, 2023
@nschonni nschonni deleted the stylelint-15 branch February 13, 2023 22:10
@mcking65 mcking65 added Infrastructure Related to maintaining task force and repo operations, processes, systems, documentation dependencies Pull requests that update a dependency file in a third-party package labels Jun 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file in a third-party package Infrastructure Related to maintaining task force and repo operations, processes, systems, documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants