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

ci: Splits test execution and coverage reporting #315

Closed
wants to merge 1 commit into from

Conversation

davelosert
Copy link
Owner

@davelosert davelosert commented Dec 16, 2023

This pull request primarily focuses on the reconfiguration of the GitHub Actions workflows to improve the coverage reporting process. The most significant changes involve the creation of a new workflow, Report Coverage, and the modification of the test.yml workflow to upload coverage reports as artifacts.

Here are the key changes:

New Workflow Creation:

  • .github/workflows/report-coverage.yml: A new workflow named Report Coverage has been introduced. This workflow triggers upon the completion of the Build and Test workflow and is responsible for downloading the coverage reports generated during testing.

Workflow Modification:

  • .github/workflows/test.yml: The Test Working Directory Option and Test Default Action steps have been commented out. Instead, a new step named Upload Coverage has been added. This step uses the upload-artifact action to upload the coverage reports generated during testing, which can then be downloaded by the Report Coverage workflow.

@davelosert
Copy link
Owner Author

This was actually just a test for #316. As I don't receive pull requests in for this repository often enough, I don't think it's justified to actually reconfigure the test-workflow to allow comments from forks.

@davelosert davelosert closed this Dec 17, 2023
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.

1 participant