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.

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

Add a report comment to PRs #108

Merged
merged 9 commits into from
Oct 30, 2023
Merged

Add a report comment to PRs #108

merged 9 commits into from
Oct 30, 2023

Conversation

ricardoboss
Copy link
Owner

@ricardoboss ricardoboss commented Oct 30, 2023

This PR is a cherry pick and subsequent continuation of changes to the dotnet.yml workflow from the branch issues/103-docblocks (at 5897eaf).

It aims to provide a quick an easy way to get a broad overview of the impact of a pull request by adding a comment to the PR containing information such as the updated code coverage, whether the code is properly formatted and what percentage of public methods is documented.

To provide the information for code coverage, a new PR was opened at irongut/CodeCoverageSummary so it can provide information about coverage in a way GitHub actions can process it. See irongut/CodeCoverageSummary#233 for more info. For the time being, a fork of that repository that already includes the PR is used (see https://github.com/ricardoboss/CodeCoverageSummary/tree/dev).

@github-actions
Copy link

github-actions bot commented Oct 30, 2023

Report

Code Coverage

Static Analysis:

  • Code Style: ✅ OK
  • Docblocks: 55 / 619 (8%)

@ricardoboss ricardoboss marked this pull request as ready for review October 30, 2023 02:08
@ricardoboss ricardoboss merged commit 570b8f5 into main Oct 30, 2023
13 checks passed
@ricardoboss ricardoboss deleted the report-comment branch October 30, 2023 14:15
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.

None yet

1 participant