search

uploadcare / uploadcare-ruby

Ruby API client that handles uploads and further operations with files by wrapping Uploadcare Upload and REST APIs.
https://uploadcare.com
MIT License
39 stars 28 forks source link

Remove PR test coverage check #51

Closed vizvamitra closed 2 years ago

vizvamitra commented 6 years ago

@dmitry-mukhin it is useless to check test coverage metrics. 100% coverage means neither good code base nor healthy test suite. Lets remove coverage check for PRs

dmitry-mukhin commented 6 years ago

I don't think we're paying excessive attention to those. Hence there is some value in those.

vizvamitra commented 6 years ago

@dmitry-mukhin but when they fail the PR is showed as broken

dmitry-mukhin commented 6 years ago

only one check and you can safely ignore it, if you know what you're doing.