-
-
Notifications
You must be signed in to change notification settings - Fork 109
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
Removes is-ci, pinst #177
Removes is-ci, pinst #177
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you. LGTM except for package-lock.json
😃
This reverts commit 7188310.
Much smaller diff, as expected! I'll squash-and-merge on approval so the history isn't bloated. |
Thanks for the thorough review! Just tested locally one more time with a fresh install on some dummy files, and it works as intended! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! LGTM 🥰
Follows the conversation in #175, to remove
is-ci
andpinst
in favour of using a shell script check andprepare
respectively.Tested locally with a dummy change and works as intended; only thing left is to check CI compat, but we've done this change across the stylelint org so I don't anticipate any issues!