Skip to content
This repository has been archived by the owner on Oct 16, 2022. It is now read-only.

Add brief Sage instructions to Readme #5

Merged
merged 1 commit into from
Apr 30, 2015

Conversation

heyfletch
Copy link
Contributor

If these minimum steps aren't done, Sage has no styling.

@retlehs
Copy link
Sponsor Member

retlehs commented Apr 27, 2015

@swalkinshaw
Copy link
Member

Agreed with @retlehs.

Having this documentation here is just another place it can get out of sync :(

@heyfletch
Copy link
Contributor Author

I agree there's potential it can get out of sync, but that's a small price to pay to make these instructions easier. There's already a lot of "go over there and read all those instructions and be sure you do everything correct and come back".

If the point of this project is to get someone up and running with all 4 projects fairly quickly, then it's tough to be continually interrupted in the flow to go and read and understand all the various parts, and then still not be sure what needs to be done given all the other pieces of the puzzle. E.g., Ansible does a lot... will it run these build commands too? Since it's not clear, might as well make it explicit here.

Those 3 build steps are necessary for Sage to work, so I think they need to be included here.

If these build steps aren't run, Sage will not render after provisioning (`vagrant up`).
Helps first-timers know they need to run these steps to get the theme to work.
@heyfletch
Copy link
Contributor Author

Updated the reference link per @retlehs

swalkinshaw added a commit that referenced this pull request Apr 30, 2015
Add brief Sage instructions to Readme
@swalkinshaw swalkinshaw merged commit c0eabf2 into roots:readme-instructions Apr 30, 2015
@swalkinshaw
Copy link
Member

Good compromise 👍

@heyfletch heyfletch deleted the patch-1 branch April 30, 2015 20:30
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