-
Notifications
You must be signed in to change notification settings - Fork 154
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 README for 4.0.1 #165
Conversation
# Installation | ||
|
||
Coverband should be near zero config for Rails apps, if you have an issue with that, please [file an issue](https://github.com/danmayer/coverband/issues). | ||
## Redis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thought we could use mention of redis a bit earlier since it is required.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense
@@ -132,18 +104,6 @@ end | |||
|
|||
Take Coverband for a spin on the live Heroku deployed [Coverband Demo](https://coverband-demo.herokuapp.com/). The [full source code for the demo](https://github.com/danmayer/coverband_demo) is available to help with installation, configuration, and understanding of basic usage. | |||
|
|||
# How To Use |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Feel like this section might be not needed anymore as it is covered elsewhere.
@@ -185,6 +145,8 @@ You may need to configure Coverband you can either do that passing in all config | |||
```ruby | |||
#config/coverband.rb | |||
Coverband.configure do |config| | |||
config.store = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['MY_REDIS_URL'])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added some more typical configuration settings since now s3 is something that will most often not be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense
This should auto-open in your browser, but if it doesn't the output file should be in `coverage/index.html` | ||
|
||
## Viewing the Report in App | ||
# Coverage Report |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tried to simplify the code example here.
README.md
Outdated
<a href="#coverband-demo">Coverband Demo</a> • | ||
<a href="#how-to-use">How To Use</a> • | ||
<a href="#coverage-report">Coverage Report</a> • | ||
<a href="#coverage-report">Coverage Report</a> • |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
duplicate coverage report line?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Whoops! Good catch!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
great nice simplifications... sorry for slow review traveling for work this week.
# Installation | ||
|
||
Coverband should be near zero config for Rails apps, if you have an issue with that, please [file an issue](https://github.com/danmayer/coverband/issues). | ||
## Redis |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense
@@ -185,6 +145,8 @@ You may need to configure Coverband you can either do that passing in all config | |||
```ruby | |||
#config/coverband.rb | |||
Coverband.configure do |config| | |||
config.store = Coverband::Adapters::RedisStore.new(Redis.new(url: ENV['MY_REDIS_URL'])) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
makes sense
f3259d3
to
707579d
Compare
No description provided.