-
-
Notifications
You must be signed in to change notification settings - Fork 6
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
Update self-test to automatically test against more options of the reusable workflow #36
Conversation
<Let's see how it goes, not sure if things will just work right off the bat> |
.github/workflows/self-test.yml
Outdated
with_coverage: | ||
type: boolean | ||
required: false | ||
default: true | ||
description: "Set to 'true' to collect and upload code coverage data. Defaults to 'false'." | ||
with_tsan: | ||
type: boolean | ||
required: false | ||
default: true | ||
description: "Set to 'true' to run tests with Thread Sanitizer. Defaults to 'true'." | ||
warnings_as_errors: | ||
type: boolean | ||
required: false | ||
default: false | ||
description: "Set to 'true' to treat warnings as errors. Defaults to 'false'." | ||
with_api_check: | ||
type: boolean | ||
required: false | ||
default: true | ||
description: "Set to 'true' to run the SwiftPM public API breakage check. Defaults to 'true'." | ||
with_gh_codeql: | ||
type: boolean | ||
required: false | ||
default: true | ||
description: "Set to 'true' to run CodeQL checks. Defaults to 'true'." | ||
with_linting: | ||
type: boolean | ||
required: false | ||
default: false | ||
description: "Set to 'true' to run swift-format's lint process. Defaults to 'false'." |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove these? These are inputs to the workflow_dispatch
event, which means "when manually run from the GitHub UI". Just change the defaults.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
emm right i guess
No description provided.