-
Notifications
You must be signed in to change notification settings - Fork 13
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
Fix #159 #164
Fix #159 #164
Conversation
Whoa, awesome! I'll look at this soon. |
This hosts TESTING.md, but Firefox prompts me to download the file or choose an application to open it with. Can we render the Markdown like we do with README.md? We currently use Gulp to convert the Markdown to HTML, but I'm not sure if that will scale now that we need to host multiple Markdown files. Make we can use the |
Sounds like an option. Gulp can be set up to scale for as many files as we On Fri, Dec 18, 2015, 16:20 John Karahalis notifications@github.com wrote:
Q: Why is this email five sentences or less? |
Also, gulp would be more efficient with more users. On Fri, Dec 18, 2015, 16:49 Blixa Morgan blixa@projectmakeit.com wrote:
Q: Why is this email five sentences or less? |
Yeah, Gulp is an option too. Whatever makes it easiest to work with multiple Markdown files. |
See #159 (comment) ... I think we should close this without fixing ... |
Closing this without fixing. We shouldn't serve the |
Using express.static should pass requests to
/docs
to the local/docs
folder.