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

Some proposal fixes #10

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

Conversation

GianniBortoloBossini
Copy link

Working, on Windows, with your amazing node version of Slate, I found these issues:

  • the deploy npm script cannot start with ./
  • the gulp steps that write into build folder have to depend on clean step to work propertly. Otherwise build and deploy sometimes fails because the folder is not empty
  • I renamed the actual markdown warning box in error and I created a new yellow warning box

@DerekTBrown
Copy link

@GianniBortoloBossini Might wanna look at this (https://github.com/mpociot/whiteboard) too

@GianniBortoloBossini
Copy link
Author

GianniBortoloBossini commented Jun 17, 2017 via email

thedeedawg referenced this pull request in Cludo/Cludo.Docs Oct 1, 2018
dpilafian added a commit to center-key/node-slate that referenced this pull request Feb 4, 2019
<aside class=error>
Error message goes here.
</aside>

Idea taken from:
sdelements#10
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