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

workflows: add a wrapper job to report test results #286

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

rafiss
Copy link
Contributor

@rafiss rafiss commented Oct 3, 2023

This wrapper job will have a stable name, so it's easy to configure a required check on it.

@rafiss rafiss requested a review from timgraham October 3, 2023 16:40
@@ -8,7 +8,22 @@ concurrency:
cancel-in-progress: true

jobs:
python38:
# Since the name of the matrix job depends on the version, we define another job with a more stable name.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would say, # This job aggregates all matrix results and is used for a GitHub required status check.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

This wrapper job will have a stable name, so it's easy to configure a
required check on it.
@rafiss rafiss merged commit 8465361 into master Oct 4, 2023
7 checks passed
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