Skip to content
This repository was archived by the owner on Jan 24, 2025. It is now read-only.

Conversation

@tagliala
Copy link
Contributor

No description provided.

aq1018 and others added 30 commits December 27, 2017 19:49
Use ruby's standard library to resolve relative path. This fixes an issue where
`Coverall.root` matches multiple parts of the `filename` string, and replacing them
with `.`. One example is `/app/app/controllers/applications_controller` would be converted to
`../controller/.lications_controller`.
…iginal_repo

drop 1.8 support, and move some work over from main coveralls
Fix relative path of filenames when root path has multiple matches.
Remove .bundle folder info from SimpleCov reports
Ch4s3 and others added 28 commits August 10, 2018 10:32
This linter could lead to misleading code coverage results

Example:

```
# bad
if condition
  do_stuff(bar)
end

# good
do_stuff(bar) if condition
```

When using the approach suggested by RuboCop, `do_something` will be
highlighted by code coverage tools, even if the method has not been
called.

Ref: https://www.rubydoc.info/gems/rubocop/RuboCop/Cop/Style/IfUnlessModifier
The old implementation of `silence_stream` has been removed from Rails.

This commit replaces the old `silence_stream` implementation with the
new one that we can actually find in Active Support.

Refs:
- rails/rails@481e49c
- rails/rails#13392
…-method

Use Active Support’s silence_stream implementation
Tests against Ruby 2.6 are going to fail.

May be related to bblimke/webmock#751
* relax-bundler-dependency:
  Relax bundler dependency
The previous code didn’t return true when the first condition was true
* fix-specs:
  Increase coverage
  Test flag name
  Fix filtering of spec folder
@tagliala tagliala closed this Mar 14, 2019
@tagliala tagliala deleted the bump-up-gems-and-rubies branch April 27, 2019 13:53
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants