Skip to content
This repository has been archived by the owner on May 8, 2019. It is now read-only.

add npm clean script to prevent error on build #8

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

wad3g
Copy link

@wad3g wad3g commented May 5, 2017

Added clean script to remove all files in the build/* directory.

This fixes issue #7 .

@rsp
Copy link

rsp commented Dec 20, 2017

@wad3g Any reason why this is not merged yet?

dpilafian referenced this pull request in center-key/node-slate Dec 23, 2017
@wad3g
Copy link
Author

wad3g commented Dec 26, 2017

@rsp No idea. Looks like the only changes the last few years has been just updating dependencies.

I stopped using this port not too long after my PR so I hadn't even noticed. Wanted to get away from Gulp in favor of plain ol' PostCSS & NPM Scripts.

dpilafian referenced this pull request in center-key/node-slate Dec 26, 2017
@rsp
Copy link

rsp commented Dec 27, 2017

Wanted to get away from Gulp in favor of plain ol' PostCSS & NPM Scripts.

@wad3g Interesting - have you managed to port node-slate gulp scripts to plain npm scripts or are you using something different for your docs now?

@wad3g
Copy link
Author

wad3g commented Dec 27, 2017

@rsp I built this for a single project so I haven't used or built it month so my memory's a little fuzzy on exactly where things left off.

I had everything working correctly using just npm scripts except for the last, and largest, hurdle -- syntax highlighting. Most of the scripts I used were just from this small/lean boilerplate I halfway put together a while back called spar here's the link to it's package.json if you'd like to see the scripts.

The docs project was on a time crunch so I had to take a few shortcuts and just use npm along side of Gulp and switching out gulp-sass for gulp-postcss. Intended to finish the npm scripts port in my freetime, but you know how that goes.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants