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

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 .

@funkaoshi funkaoshi requested a review from sibartlett June 19, 2017 21:24
@rsp
Copy link

rsp commented Dec 20, 2017

@wad3g Any reason why this is not merged yet?

pilafmon 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.

pilafmon 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.

3 participants