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

Clear documentation on blacksmith requirements can reduce invalid bugs along the road #85

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

Conversation

taboca
Copy link

@taboca taboca commented Jan 18, 2013

I have patched your documentation changing slightly the sections so it is clear that it is important to have a directory structure before executing "blacksmith /path/to/site" I have noticed about 3 bugs that I feel may have existed because of the docs problem:

#83
#79
#69

@cronopio
Copy link

cronopio commented Feb 3, 2013

LGTM

@cbebry
Copy link

cbebry commented May 17, 2013

+1
I was about to make another issue if I didn't find others with the same problem -- as I encountered this as well.

It's been 4 months since this pull request. Is there a reason why it isn't merged or addressed? Or made clearer in the documentation?

@koenmetsu
Copy link

I'd love to try out Blacksmith, but the documentation leaves me with a lot of questions. Like a couple of others in the issues page, I haven't yet been able to make this work because I figured Blacksmith would generate the necessary files + some example pages.

This open pull request makes me wonder if this project is still alive at all :(

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.

5 participants