Skip to content

minor formatting for README files#48

Merged
johnkim76 merged 2 commits into
devfrom
doc_update
May 26, 2017
Merged

minor formatting for README files#48
johnkim76 merged 2 commits into
devfrom
doc_update

Conversation

@johnkim76
Copy link
Copy Markdown
Contributor

No description provided.

@johnkim76 johnkim76 changed the title minor formatting for README files [WIP DO NOT MERGE] - minor formatting for README files May 17, 2017
@johnkim76 johnkim76 changed the title [WIP DO NOT MERGE] - minor formatting for README files minor formatting for README files May 17, 2017
Copy link
Copy Markdown
Contributor

@jmrodri jmrodri left a comment

Choose a reason for hiding this comment

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

Make sure all of the commands that need to get run are escaped with the tick marks.

Comment thread ec2/README.md Outdated
2.2.2.0
* Install python dependencies (This is needed for python2. Use pip2 if using python3)
* `pip install boto boto3 six`
$ pip install boto boto3 six
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

This needs to be marked with ``` you can't see it in the formatted view.

  • Install python dependencies (...)
$ pip install boto boto3 six

something like that

Comment thread ec2/README.md
$ ./run_create_infrastructure.sh
* Run the setup script
$ ./run_setup_environment.sh
* Open a Web Browser
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

You're adding a newline to try to make it easier to read, but it doesn't work. It just ends up being on the same line. I would make all the commands escaped with triple tick marks ```

Comment thread ec2/README.md
* Accept the SSL certificate for the apiserver-service-catalog endpoint
* Ignore the text that appears and proceed to the main OpenShift URL next
* Note: must accept the new SSL cert, each time you reset your OpenShift environment
* Visit: `https://<USERNAME>.ec2.dog8code.com:8443`
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No changes here.

Comment thread local/README.md Outdated
* `cd local`
* `./reset_environment.sh`
* To reset the environment to a clean instance of origin with ASB and Service Catalog run the below
$ ./reset_environment.sh
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Again, these commands get butted to the end of the sentences and you can't really see they are commands that need executing.

encapsulate shell commands and examples with a ```
@johnkim76
Copy link
Copy Markdown
Contributor Author

@jmrodri thank you. I've made the changes you suggested.

@johnkim76 johnkim76 merged commit f74b511 into dev May 26, 2017
@johnkim76 johnkim76 deleted the doc_update branch June 14, 2017 13:05
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