Issue 1218: Remove codecov from PR check list

[Jonatan-Chaverri]

Closes #1218

Introduced changes

• Disable the Codecov status checks on PRs. This means that Codecov will no longer block PRs from being merged due to coverage decreases or not meeting a certain coverage threshold.
• Allow Codecov to continue posting comments on PRs with the coverage report. This means that even though Codecov won't block PRs due to coverage issues, you'll still be able to see the impact of the PR on the code coverage in the Codecov comment.

• This PR contains breaking changes

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (75c6b8a) 97.95% compared to head (7c4c169) 97.31%.

Additional details and impacted files

@@               Coverage Diff               @@
##           development    #1258      +/-   ##
===============================================
- Coverage        97.95%   97.31%   -0.65%     
===============================================
  Files               89       89              
  Lines             4506     4506              
===============================================
- Hits              4414     4385      -29     
- Misses              92      121      +29     

see 6 files with indirect coverage changes

[Jonatan-Chaverri]

Not sure why tests are failing 🤔 I didn't modify any logic... let me know if there is anything else I need to do.

[Jonatan-Chaverri]

@ddoktorski Done! Please review again. Thanks.

[ddoktorski]

Thank you for your contribution!

This looks good to me. The failing tests are most likely due to unset secrets for the environment variables in your forked repository. That's fine in this case.

Please feel free to check out other items from the issues list and let us know if you find something interesting to work on :)