-
Notifications
You must be signed in to change notification settings - Fork 100
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
Remove ui-stylelint-config, update dependecies #1707
Conversation
|
{null} | ||
{false} |
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.
new TS version is clever and figures out that the part after &&
is never executed
@@ -52,10 +52,11 @@ export default { | |||
|
|||
// optimize svgs in place | |||
runCommandSync('svgo', [ | |||
'-r', | |||
'-f', | |||
'--quiet', |
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.
this will make bootstrap
a bit less spammy
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.
great work!
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.
Nice!
It was used to lint .css files, which is kinda useless by itself because we only have a few .css files and most of them are autogenerated
Remove
ui-stylelint-config
because:.css
files, which is kinda useless by itself because we only have a few.css
files and most of them are autogenerated.lint
command because we did not call ourlint
command on the few packages that have.css
files..css
file.Also updated dependencies, mostly minor version updates