-
Notifications
You must be signed in to change notification settings - Fork 32
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
remove CodeCov from test suite #172
Conversation
Also remove the main Gemfile.lock to allow gem versions to float to latest resolveable by gemspec and Ruby version constraints.
Because Gemfile.lock has been removed to allow dependencies to resolve to latest possible, we'll cache dependencies retrieved based on changes to the gemspec instead of the lock file. Good 'nuff.
require "webmock/rspec" | ||
require "pry" | ||
|
||
WebMock.disable_net_connect! | ||
|
||
Dir["./spec/support/**/*.rb"].sort.each { |f| require f } | ||
SimpleCov.formatter = SimpleCov::Formatter::Console | ||
SimpleCov.formatter = SimpleCov::Formatter::Codecov if SUPPORT_CODECOV |
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.
why remove this line only?
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.
It's the line that configures the coverage generating tool (SimpleCov) to use the formatter that sends the report to CodeCov. Without this line, the previous line remains in effect: report current code coverage to STDOUT.
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.
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.
Remove CodeCov. (Closes #168.) We're not tracking coverage overtime as much as we used to. SimpleCov is still available to help determine if there is sufficient coverage of a change.
Also remove the main Gemfile.lock (Closes #170.) to allow gem versions to float to latest resolveable by gemspec and Ruby version constraints.