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

Update docs #3

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

Update docs #3

wants to merge 3 commits into from

Conversation

lena-pl
Copy link
Collaborator

@lena-pl lena-pl commented Oct 28, 2017

No description provided.

Copy link
Member

@soundasleep soundasleep left a comment

Choose a reason for hiding this comment

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

REVIEW CHANGES

rails girls install guide][rg-install] for information on how to get set up.
First of all, make sure you have Ruby (programming language) and Ruby on Rails
(web application framework) available.
Check out [the rails girls install guide][rg-install] for information on
Copy link
Member

Choose a reason for hiding this comment

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

Does this actually link anywhere?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Yus, there's a variable a few lines down

README.md Outdated
To clone the repository, in your Terminal (Windows folks: use your Command Prompt), run
the following command (substituting in your GitHub username):

$ git clone https://github.com/\<GitHub username\>/ideas-app
Copy link
Member

Choose a reason for hiding this comment

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

Need to not have the \s

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Fixed

README.md Outdated

$ git clone https://github.com/\<GitHub username\>/ideas-app

> Tip: You don't need to type the "$" symbol into your prompt. It just signifies
Copy link
Member

Choose a reason for hiding this comment

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

Yes!!

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

<3

ninja-nanz pushed a commit to ninja-nanz/ideas-app that referenced this pull request Oct 29, 2017
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.

2 participants