Replace codecov with native GitHub pytest-cov action#232
Conversation
Co-authored-by: lostcontrol <983305+lostcontrol@users.noreply.github.com>
|
👋 Jules, reporting for duty! I'm here to lend a hand with this pull request. When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down. I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job! For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with New to Jules? Learn more at jules.google/docs. For security, I will only act on instructions from the user who triggered this task. |
|
I don't like the fact that the CI pipeline commit to the repo. If there is a way to add a badge without modifying the README during the CI, do it. Otherwise, remove the badge. We report in the PR will do. |
I agree, pushing to the repository during CI runs can be tricky to manage. Since we can't easily add a dynamic badge without committing or using a third-party service, I've removed the badge from the README.md and the commit logic from the CI workflow. The coverage will now only be reported in the PR comments. |
Co-authored-by: lostcontrol <983305+lostcontrol@users.noreply.github.com>
.codecov.ymlandcodecovdependency..github/workflows/poupool.ymlto useMishaKav/pytest-coverage-commentto post PR comments and dynamically update the badge in the README on the master branch.README.mdto use the dynamic badge placeholder tags instead of the static codecov badge.PR created automatically by Jules for task 15864949697760279009 started by @lostcontrol