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

[test] draft pr - DO NOT MERGE #413

Closed
wants to merge 2 commits into from
Closed

[test] draft pr - DO NOT MERGE #413

wants to merge 2 commits into from

Conversation

JP-Ellis
Copy link
Contributor

@JP-Ellis JP-Ellis commented Oct 8, 2023

📝 Summary

This is a test draft pull request.

🚨 Breaking Changes

🔥 Motivation

🔨 Test Plan

🔗 Related issues/PRs

As draft PRs are explicitly in a state which is not ready to be merged,
there is no need to run the full suite of CI workflows every time there
is an update. In particular, this commit disables:

- Cirrus workflows, as Cirrus has a minute quota
- Wheel builds
- Long-running tests

The test matrix is _still_ being run.

Signed-off-by: JP-Ellis <josh@jpellis.me>
The previous configuration for concurrency group used `head_ref`
incorrectly resulting in separate runs on master being cancelled.

This solution was taken from: https://stackoverflow.com/a/75403978/1573761

Signed-off-by: JP-Ellis <josh@jpellis.me>
@JP-Ellis JP-Ellis changed the title [test] draft pr [test] draft pr - DO NOT MERGE Oct 10, 2023
@JP-Ellis JP-Ellis self-assigned this Oct 10, 2023
@JP-Ellis
Copy link
Contributor Author

Closed as the PR served as a test.

@JP-Ellis JP-Ellis closed this Oct 11, 2023
@JP-Ellis JP-Ellis deleted the ci/draft-pr-test branch October 12, 2023 02:36
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