AutoGPT icon indicating copy to clipboard operation
AutoGPT copied to clipboard

[Hotfix] Fix coverage tooling

Open ntindle opened this issue 2 years ago • 3 comments

Background

Minor issue with CI failing due to push events trying to upload coverage

Changes

Documentation

Test Plan

PR Quality Checklist

  • [x] My pull request is atomic and focuses on a single change.
  • [x] I have thoroughly tested my changes with multiple different prompts.
  • [x] I have considered potential risks and mitigations for my changes.
  • [x] I have documented my changes clearly and comprehensively.
  • [x] I have not snuck in any "extra" small tweaks changes

ntindle avatar Apr 21 '23 05:04 ntindle

I hate that this is size/l

ntindle avatar Apr 21 '23 06:04 ntindle

As mentioned in the original issue, we could just install Codecov. Easier to configure and doesn't require the same extensive permissions.

Pwuts avatar Apr 21 '23 10:04 Pwuts

Someone with full repo control would need to set that up and I’m not sure if it’s free

ntindle avatar Apr 21 '23 13:04 ntindle

CodeCov appears to be free for OSS projects. We do need to fix the CI @Pwuts are you happy for this to be merged and we can check out codecov at a later date?

richbeales avatar Apr 21 '23 18:04 richbeales

If I get perms to add code cov, I’d love to. Otherwise I’d just like to get this working

ntindle avatar Apr 21 '23 18:04 ntindle

merged, better to fix CI now rather than wait for an unknown amount of time for a better solution.

richbeales avatar Apr 21 '23 18:04 richbeales