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

Simplify README documentation #90

Merged
merged 3 commits into from
Feb 25, 2018
Merged

Simplify README documentation #90

merged 3 commits into from
Feb 25, 2018

Conversation

bobheadxi
Copy link
Member

βŒ› Status: Ready

🎟️ Ticket(s): #88


πŸ‘· Changes

Simplified setup documentation in the README and reduced the size of the logo slightly

πŸ”¦ Testing Instructions

n/a

@bobheadxi bobheadxi added the pr: finalized needs review and final approval label Feb 24, 2018
@coveralls
Copy link

coveralls commented Feb 24, 2018

Coverage Status

Coverage remained the same at 25.543% when pulling 389c525 on rob/#88-readme-update into 6855493 on master.

Copy link
Contributor

@chadlagore chadlagore left a comment

Choose a reason for hiding this comment

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

LGTM πŸ‘

README.md Outdated
```

## Development
The Inertia daemon requires readonly access to your GitHub repository. Add it to your GitHub repository settings at the URL provided in the output.

Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe add this note to the (2) below

README.md Outdated
If you make changes to the bootstrapping shell scripts in
`client/bootstrap/`, convert them to `Assets` by running:
$> inertia my-remote-vps status
# Reports detailed information about
Copy link
Contributor

Choose a reason for hiding this comment

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

About what?😝

Copy link
Member Author

Choose a reason for hiding this comment

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

that's the mystery

@bobheadxi bobheadxi merged commit 9151683 into master Feb 25, 2018
@bobheadxi bobheadxi deleted the rob/#88-readme-update branch February 25, 2018 00:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: finalized needs review and final approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants