Closed agarzola closed 2 years ago
This PR updates the styles task to remove a regression that was causing it to catch Sass compilation errors and, therefore, fail silently.
styles
Discovered while attempting to test styles task for #22.
Tested locally during a pairing session with @michelegrace.
Sass compilation error causes the process to exit with a thrown error on yarn build:
yarn build
Sass compilation error is logged, but does not cause the process to exit on yarn watch:
yarn watch
I added a comment to the error handling in question.
Description
This PR updates the
styles
task to remove a regression that was causing it to catch Sass compilation errors and, therefore, fail silently.Motivation / Context
Discovered while attempting to test
styles
task for #22.Testing Instructions / How This Has Been Tested
Tested locally during a pairing session with @michelegrace.
Screenshots
Sass compilation error causes the process to exit with a thrown error on
yarn build
:Sass compilation error is logged, but does not cause the process to exit on
yarn watch
:Documentation
I added a comment to the error handling in question.