Skip to content

Add an option to generate a plain-text markdown table for PR coverage#639

Closed
dvg-p4 wants to merge 3 commits intopy-cov-action:mainfrom
dvg-p4:dvg/plain-text-pr-coverage
Closed

Add an option to generate a plain-text markdown table for PR coverage#639
dvg-p4 wants to merge 3 commits intopy-cov-action:mainfrom
dvg-p4:dvg/plain-text-pr-coverage

Conversation

@dvg-p4
Copy link

@dvg-p4 dvg-p4 commented Feb 28, 2026

No description provided.

@github-actions
Copy link

End-to-end public repo

Admin commands cheatsheet:

  • /e2e (in approved PR review body): Trigger end-to-end tests on external contributions
  • /invite (in comment): Invite the author & admins to the end-to-end private repo

@github-actions
Copy link

github-actions bot commented Feb 28, 2026

Coverage report

Click to see where and how coverage changed

FileStatementsMissingCoverageCoverage
(new stmts)
Lines missing
  coverage_comment
  main.py 210-223
  settings.py 101
  template.py 243-244
Project Total  

This report was generated by python-coverage-comment-action

@dvg-p4 dvg-p4 force-pushed the dvg/plain-text-pr-coverage branch from fdfba68 to 79c2d6d Compare March 1, 2026 00:59
@ewjoachim
Copy link
Member

Hello,

The report as it stands today almost reaches the character limits for github comments already.
BTW, you can already override the template if you want.

Maybe I'm missing an important point, in which case feel free to raise it. I don't like deying a contribution on which you've worked, independently of its quality, which is why it's best to open an issue for discussing this first.

@dvg-p4
Copy link
Author

dvg-p4 commented Mar 11, 2026

Oh don't waste your time taking a look at this yet, it's pure vibecoded prototype at the moment. I was just trying to test it on itself, which didn't really work (I missed the fact that there was a whole test repo setup...facepalm)

...yeah using the built-in override functionality would probably be simpler here too, d'oh.

@dvg-p4 dvg-p4 closed this Mar 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants