-
Notifications
You must be signed in to change notification settings - Fork 662
docs: enable_workflow_job_labels_check -> enable_runner_workflow_job_labels_check_all #3665
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
Conversation
124639b
to
d4180c0
Compare
With the PR, However,
Its only appearance is in the root level variables.tf and main.tf. Do those apply in /examples/multi-runner/ or modules/multi-runner/? Correct me if this is wrong. I think what should happen, is to slightly refactor the code of multi-runner so that it uses the variable |
@sdarwin I hope to have tomorrow some time to dig in the label part. It certainly require some owrk. And evenutally I think some reafactoring. |
Did I fat finger close the PR? @npalm |
BTW it seems it is mentioned in variables only in the description as it actually take it from another place @sdarwin |
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.
LGTM, thx!
@npalm which place? The variable "enable_runner_workflow_job_labels_check_all" is defined in the root level, top level, files https://github.com/philips-labs/terraform-aws-github-runner/blob/main/main.tf Does multi-runner reference those files? |
There are two possibilities:
|
enable_workflow_job_labels_check is no longer exist, correct?
and now enable_runner_workflow_job_labels_check_all is the functioning variable, correct?
This is just to swap in the READMEs and samples