Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Better format for coverage comment #42

Closed
acocheo opened this issue Jul 1, 2022 · 3 comments
Closed

Better format for coverage comment #42

acocheo opened this issue Jul 1, 2022 · 3 comments
Assignees
@acocheo
Labels
bug Something isn't working external Not strictly related with this repository

Comments

@acocheo
Copy link
Collaborator

acocheo commented Jul 1, 2022

Tables need tabs align
@acocheo acocheo changed the title Better format coverage comment Better format for coverage comment Jul 1, 2022
@acocheo acocheo added the bug Something isn't working label Jul 1, 2022
@0ssigeno 0ssigeno added the frontend Issue involving HTML, Node etc... label Jul 4, 2022
@acocheo
Copy link
Collaborator Author

acocheo commented Jul 8, 2022

Instead of comments, we can use Job summaries (they support markdown)
https://github.blog/2022-05-09-supercharging-github-actions-with-job-summaries/
https://docs.github.com/en/actions/using-workflows/workflow-commands-for-github-actions#adding-a-job-summary

@acocheo acocheo removed the frontend Issue involving HTML, Node etc... label Jul 8, 2022
@acocheo acocheo self-assigned this Jul 18, 2022
acocheo added a commit that referenced this issue Jul 19, 2022
@acocheo
Coverage comment #42
458c31f
@acocheo
Copy link
Collaborator Author

acocheo commented Jul 19, 2022

Waiting this feature for coverage.py

acocheo added a commit that referenced this issue Jul 19, 2022
@acocheo
Coverage.py comment #42
8c7eaab
@acocheo acocheo mentioned this issue Jul 19, 2022
Merged
@acocheo acocheo added the external Not strictly related with this repository label Jul 19, 2022
0ssigeno added a commit that referenced this issue Jul 25, 2022
@0ssigeno
Merge pull request #80 from certego/step-summary
a341145 
Coverage comment #42
@acocheo
Copy link
Collaborator Author

acocheo commented Sep 6, 2022

Duplicate #95

@acocheo acocheo mentioned this issue Sep 6, 2022
Closed
acocheo added a commit that referenced this issue Dec 27, 2022
@acocheo
Update deps + Fix #42
5c51aee
acocheo added a commit that referenced this issue Dec 27, 2022
@acocheo
Update deps + Fix #42
5fb97ec
@acocheo acocheo closed this as completed Dec 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@acocheo acocheo

Labels
bug Something isn't working external Not strictly related with this repository
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@0ssigeno @acocheo