ci:testing:remove codecov

[ZenGround0]

Related Issues

Proposed Changes

It's been a good run, it doesn't appear that this is helping much. So here's an experiment to remove codecov and see if it makes our lives easier. We can always revert if we miss it.

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

• All commits have a clear commit message.
• The PR title is in the form of of <PR type>: <area>: <change being made>
  • example: fix: mempool: Introduce a cache for valid signatures
  • PR type: fix, feat, INTERFACE BREAKING CHANGE, CONSENSUS BREAKING, build, chore, ci, docs,perf, refactor, revert, style, test
  • area: api, chain, state, vm, data transfer, market, mempool, message, block production, multisig, networking, paychan, proving, sealing, wallet, deps
• This PR has tests for new functionality or change in behaviour
• If new user-facing features are introduced, clear usage guidelines and / or documentation updates should be included in https://lotus.filecoin.io or Discussion Tutorials.
• CI is green

[arajasek]

Glad we did it, glad to be undoing it.

[jennijuju]

It was a good run, It did the purpose of yelling "YOU NEED A TEST" at us, and I can def see an improvement in having tests with PRs from @filecoin-project/lotus-maintainers.

Maintainers must continue requesting tests for PRs that introduce new functionality or change in behavior and only merge PR to master that has good test coverage.

Approving cuz also agree at this point it's not helping much and sometimes gives annoying ❌ that we ended up ignoring anyways (bad blocker for "enable multi-merge" or ci)