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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix scss watch tasks broken on syntax error #3445

Merged
merged 1 commit into from Aug 9, 2023

Conversation

t-fritsch
Copy link
Collaborator

when there is a syntax error in a sass file (theme or core) the npm start command used to hang, forcing the user to stop and restart the task to compile again.

this fix allows to keep the start-task watching/compiling even when there is an error :

  • the error is displayed in terminal
  • the rest of gulp tasks are not called (no reload in the browser)
  • the user can edit the scss files to try a fix without the need to stop/restart the npm start command

useful when you're working on a theme and you keep making mistakes in the scss code 馃槄

when there is a syntax error in a sass file (theme or core) the npm start command used to hang, forcing the user to stop and restart the task to compile again.
this fix allows to keep the start-task watching/compiling even when there is an error :
- the error is displayed in terminal
- the rest of gulp tasks are not called (no reload in the browser)
- the user can edit the scss files to try a fix without the need to stop/restart the `npm start` command
@hakimel hakimel merged commit e1c1805 into hakimel:master Aug 9, 2023
@hakimel
Copy link
Owner

hakimel commented Aug 9, 2023

Another great fix. Thanks!

@t-fritsch t-fritsch deleted the fix-scss-watch-tasks branch September 12, 2023 11:47
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.

None yet

2 participants