Skip to content

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