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

Require OS and Suite inputs #11

Merged
merged 2 commits into from
May 16, 2022
Merged

Require OS and Suite inputs #11

merged 2 commits into from
May 16, 2022

Conversation

damacus
Copy link
Contributor

@damacus damacus commented Nov 8, 2021

Description

If these inputs are not set then we're going to match multiple suites which is probably not the expect behaviour during CI

Issues Resolved

List any existing issues this PR resolves

Check List

  • New functionality includes testing.
  • New functionality has been documented in the README if applicable.

If these inputs are not set then we're going to match multiple suites which is probably not the expect behaviour during CI
@damacus damacus requested a review from xorima April 11, 2022 12:59
@xorima
Copy link
Contributor

xorima commented Apr 22, 2022

Make sure this is highlighted as a breaking change when merged.
People might be doing just a suite or os and running them all on a single worker for some reason

@damacus damacus merged commit 795f58a into main May 16, 2022
@damacus damacus deleted the require-inputs branch May 16, 2022 12:58
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