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

Replace scss_lint with stylelint #5641

Merged
merged 4 commits into from
Nov 29, 2021
Merged

Conversation

mitchellhenke
Copy link
Contributor

scss_lint depends on ruby-sass, which reached it's end-of-life in March of 2019, so we should drop it. stylelint seemed to be the consensus replacement in past discussions and more generally in this space.

I started with the rule set in stylelint-config-recommended-scss as it was the least restrictive. stylelint-config-standard-scss is the other option should we want to include some of the rules there.

I am not confident that we have covered all of the old rules that scss_lint had though.

Comment on lines -560 to -564
sass (3.7.4)
sass-listen (~> 4.0.0)
sass-listen (4.0.0)
rb-fsevent (~> 0.9, >= 0.9.4)
rb-inotify (~> 0.9, >= 0.9.7)
Copy link
Contributor

Choose a reason for hiding this comment

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

YESSSSS next up remove sassc!!! 🔥 🔥 🔥

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I don't know if my webpack skills are sufficient for the sprocket replacement journey, but I agree 😛

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Also guessing we'll want to do the webpack 4 -> webpack 5 thing at some point too

Copy link
Member

Choose a reason for hiding this comment

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

Also guessing we'll want to do the webpack 4 -> webpack 5 thing at some point too

Still waiting on Webpacker v6 😩 (rc6 as of a few weeks ago)

@mitchellhenke mitchellhenke marked this pull request as ready for review November 24, 2021 21:30
Copy link
Member

@aduth aduth left a comment

Choose a reason for hiding this comment

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

Right now this doesn't seem to enforce much at all in terms of styling. For example, the following is perfectly valid:

.language-desktop-button{

  background-color:transparent; }

The other configuration you mentioned seems interesting as enforcing some common conventions. Did you try it out?

The 18F Engineering Guide codifies some of the styling we might want to aim for if we'd care to build out our own configuration.

I'm also fine to just get this in to unblock Ruby 3 and sort out the specific styling rules later / as we encounter them.

package.json Show resolved Hide resolved
mitchellhenke and others added 3 commits November 29, 2021 12:58
Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>
Co-authored-by: Andrew Duthie <andrew.duthie@gsa.gov>
@mitchellhenke mitchellhenke merged commit 86dbc95 into main Nov 29, 2021
@mitchellhenke mitchellhenke deleted the mitchellhenke/stylelint branch November 29, 2021 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants