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

solved the need for 'gulp sass', restructured the directories and added an alternative gulp only build #2

Open
wants to merge 38 commits into
base: master
Choose a base branch
from

Conversation

patmuk
Copy link

@patmuk patmuk commented May 7, 2017

An update:
In my latest code I changed the build to integrate gulp tasks and even to have a nom-only build without gulp.

Well, it turns out that your initial setup (metalsmith only) is fastest.
But theoretically the gulp combination should be faster, because it does things in parallel.
The idea is: All set of files, which can be processed individually, should have an own gulp task.
I am sticking with this in my latest code.

The npm only build (as it is trendy now, because it eliminates wrapped plugins) turned out to be 2-3 times slower! I assume this is because each npm script execution has a ramp-up time.

Thought that might be interesting to you :)

Hey,

I made mayor changes, to speed up the build and make the structure clearer.

This fixes the need to run gulp sass before node .

The directory now has a clear separation of source files (in /src) and built files (generated to /build).

As an alternative to node . gulp can be used now. This is faster and more flexible.

Additional minor changes:
I added:

  • metalsmith-browser-sync (starts the compiled pages in a browser window)
  • metalsmith-debug (logs what is going on in metalsmith - metadata, files, etc, some plugins as well)

Best,

Patrick

…es to clearly seperate between sources and build output. gulp works as well.
@patmuk patmuk changed the title cosmetic changes - added browser-sync and metalsmith-debug solved the need for 'gulp sass', restructured the directories and added an alternative gulp only build May 9, 2017
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.

1 participant