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

Improve bower documentation #253

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

Conversation

djds23
Copy link
Collaborator

@djds23 djds23 commented Jan 31, 2016

Link to documentation from the main README documentation.
#151

Note: https://help.github.com/articles/relative-links-in-readmes/

@@ -101,6 +101,8 @@ The last step is to create an event::
Activate the event [n]: <y|n>


In order to get JavaScript working locally, you must follow the `front end setup instructions here`_.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please remove the word "here." It makes the sentence awkward.

@dirn
Copy link
Member

dirn commented Feb 1, 2016

Come to think of it, that README was most likely autogenerated by Compass. It's not very useful and doesn't really help people using Docker/Compass. We should probably focus on updating the main README and the Dockerfile instead.

@djds23
Copy link
Collaborator Author

djds23 commented Feb 1, 2016

@dirn I added steps to the docs. I also attempted to fix the Dockerfile. That being said I am not too familiar with docker and the code is untested, so we should not merge until someone who uses Docker has tested it out.

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.

2 participants