[GitHub] [incubator-superset] mistercrunch edited a comment on issue #8606: Using Prettier for CSS

[GitBox <gi...@apache.org>]

mistercrunch edited a comment on issue #8606: Using Prettier for CSS
URL: https://github.com/apache/incubator-superset/pull/8606#issuecomment-555846865
 
 
   Happy to merge this as-is, but do we want to break the build if there's lint in the CSS?
   
   I think to do so we'd have to add a trailing command to the npm run lint.
   
   Oh what about adding a commit hook as well?

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
 
For queries about this service, please contact Infrastructure at:
users@infra.apache.org


With regards,
Apache Git Services

---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@superset.apache.org
For additional commands, e-mail: notifications-help@superset.apache.org