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

Add allows_public_repositories to actions_runner_group #1998

Merged
merged 2 commits into from
Jul 15, 2021

Conversation

FloThinksPi
Copy link
Contributor

Adds allows_public_repositories to request bodies of the
CreateRunnerGroup as well as UpdateRunnerGroup methods for github
actions.

This allows us to set the allows_public_repositories flag even if it is
not yet properly documented see https://github.com/github/docs/issues/8209
That flag controls if public repos can use runners of this group.
If it is disabled(default) even public repos that are added to the
selected_repositories list cannot use runners of this group.

With this change we can now set this properly and thus use runner groups
created by go-github also for public repos.

Fixes #1996

Adds allows_public_repositories to request bodies of the
CreateRunnerGroup as well as UpdateRunnerGroup methods for github
actions.

This allows us to set the allows_public_repositories flag even if it is
not yet properly documented see https://github.com/github/docs/issues/8209
That flag controls if public repos can use runners of this group.
If it is disabled(default) even public repos that are added to the
selected_repositories list cannot use runners of this group.

With this change we can now set this properly and thus use runner groups
created by go-github also for public repos.

Fixes google#1996
@google-cla google-cla bot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Jul 14, 2021
@FloThinksPi
Copy link
Contributor Author

FloThinksPi commented Jul 14, 2021

I mirrored the style from the existing structs to do pass by reference so having a bool pointer.
Is this the wanted way/interface ?

Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @FloThinksPi !
Please run go generate ./... and push (not force-push) the changes to this PR, as described in our CONTRIBUTING.md document.

@FloThinksPi FloThinksPi requested a review from gmlewis July 15, 2021 07:26
Copy link
Collaborator

@gmlewis gmlewis left a comment

Choose a reason for hiding this comment

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

Thank you, @FloThinksPi !
LGTM.
Merging.

@gmlewis gmlewis merged commit b2c5204 into google:master Jul 15, 2021
@FloThinksPi FloThinksPi deleted the allows_public_repositories branch July 16, 2021 07:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indication that the PR author has signed a Google Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

We do not set allows_public_repositories for actions runner groups
2 participants