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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merge together sass watch and serve #19

Closed
wants to merge 2 commits into from

Conversation

budavariam
Copy link
Contributor

#15

Based on: https://egghead.io/lessons/11ty-add-sass-compiling-and-watch-for-changes-in-eleventy-11ty

Notes:

  • I think the pipeline should not directly modify files in the _site folder, that should handled by eleventy, thus I save the generated files to css folder, and passthroughcopy to the proper location
  • I'm not sure what netlify dev does, I hope these changes did not break the workflow
  • new dependency: npm-run-all, cross platform, seems legit by their user count
  • I do not trust the globstar in the tutorial's build:*, we have to run sass first thus "build": "npm-run-all build:sass build:eleventy", seems more reliable

@cat-a-flame
Copy link
Owner

Already got a solution for this, I just didn't had time to test it. I'll check your too, thank you!

@cat-a-flame
Copy link
Owner

Been playing around with examples like yours and this one, but they all result in a strange behavior: sometimes I need to save twice on the sass files to see the effect on my site. I'm not sure this is caused by the fact that I'm using Windows and it has permission problems, or something else.

@cat-a-flame
Copy link
Owner

After several tests I've merged my solution which is quiet similar to yours, it just required some fine tuning. Thanks for the help and sorry for not choosing this one :( :)

@cat-a-flame cat-a-flame closed this Jun 5, 2021
@budavariam
Copy link
Contributor Author

Thanks for checking it out! It was fooish of me to assume that it'd work in windows out of the box.
I'm glad you found a working solution.

@cat-a-flame
Copy link
Owner

Windows is just Windows, don't feel bad for not thinking about it :D

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