-
Notifications
You must be signed in to change notification settings - Fork 774
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
feat: Add github_actions_enterprise_permissions
#2155
Conversation
* Adding enterprise permissions resource * Working version * Adding docs * Functioning tests
When running the new integration tests, I get the following error:
Can you reproduce this? Am I missing some kind of setup? |
I missed the check on Updated the PR. |
I don't know if there is a good reason for this, but between this PR and #2158 there is some naming inconsistency: |
@laughedelic good catch. I have a slight preference for |
That was my thinking too, though there was only one example ( |
I named this after |
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.
I don't have easy access to an enterprise for manual validation so please let me know if something isn't behaving right!
Works in my enterprise. I think that's acceptable tbh.
|
This PR contains the following updates: | Package | Type | Update | Change | |---|---|---|---| | [github](https://registry.terraform.io/providers/integrations/github) ([source](https://togithub.com/integrations/terraform-provider-github)) | required_provider | minor | `6.0.1` -> `6.2.0` | --- ### Release Notes <details> <summary>integrations/terraform-provider-github (github)</summary> ### [`v6.2.0`](https://togithub.com/integrations/terraform-provider-github/releases/tag/v6.2.0) [Compare Source](https://togithub.com/integrations/terraform-provider-github/compare/v6.1.0...v6.2.0) <!-- Release notes generated using configuration in .github/release.yml at main --> #### What's Changed ##### Features - feat: add `github_enterprise_actions_runner_group` by [@​bradam12](https://togithub.com/bradam12) in [https://github.com/integrations/terraform-provider-github/pull/2158](https://togithub.com/integrations/terraform-provider-github/pull/2158) - Feat: Fixes abandoned PR [#​2017](https://togithub.com/integrations/terraform-provider-github/issues/2017): Add support for the require_last_push_approval flag in github_branch_protection_v3 by [@​georgekaz](https://togithub.com/georgekaz) in [https://github.com/integrations/terraform-provider-github/pull/2199](https://togithub.com/integrations/terraform-provider-github/pull/2199) ##### Bugfixes - fix(data_source_github_rest_api): just read body and convert bytes into string by [@​riezebosch](https://togithub.com/riezebosch) in [https://github.com/integrations/terraform-provider-github/pull/2152](https://togithub.com/integrations/terraform-provider-github/pull/2152) ##### 🛠️ Maintenance - build(deps): bump golang.org/x/oauth2 from 0.17.0 to 0.18.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/integrations/terraform-provider-github/pull/2189](https://togithub.com/integrations/terraform-provider-github/pull/2189) - build(deps): bump google.golang.org/protobuf from 1.32.0 to 1.33.0 by [@​dependabot](https://togithub.com/dependabot) in [https://github.com/integrations/terraform-provider-github/pull/2194](https://togithub.com/integrations/terraform-provider-github/pull/2194) **Full Changelog**: integrations/terraform-provider-github@v6.1.0...v6.2.0 ### [`v6.1.0`](https://togithub.com/integrations/terraform-provider-github/releases/tag/v6.1.0) [Compare Source](https://togithub.com/integrations/terraform-provider-github/compare/v6.0.1...v6.1.0) <!-- Release notes generated using configuration in .github/release.yml at main --> #### What's Changed - fix: validation rule for `results_per_page` of `github_repositories` data source by [@​dschniepp](https://togithub.com/dschniepp) in [https://github.com/integrations/terraform-provider-github/pull/2185](https://togithub.com/integrations/terraform-provider-github/pull/2185) - fix: Prevent loading of allowed actions if not configured by [@​Danielku15](https://togithub.com/Danielku15) in [https://github.com/integrations/terraform-provider-github/pull/2186](https://togithub.com/integrations/terraform-provider-github/pull/2186) - fix(data_source_github_rest_api): only allow for 404 on err by [@​riezebosch](https://togithub.com/riezebosch) in [https://github.com/integrations/terraform-provider-github/pull/2154](https://togithub.com/integrations/terraform-provider-github/pull/2154) - fix: error if autolink reference not found by [@​bradam12](https://togithub.com/bradam12) in [https://github.com/integrations/terraform-provider-github/pull/2164](https://togithub.com/integrations/terraform-provider-github/pull/2164) - feat: Add `github_actions_enterprise_permissions` by [@​ErikElkins](https://togithub.com/ErikElkins) in [https://github.com/integrations/terraform-provider-github/pull/2155](https://togithub.com/integrations/terraform-provider-github/pull/2155) - docs: configure release notes categories based on labels by [@​laughedelic](https://togithub.com/laughedelic) in [https://github.com/integrations/terraform-provider-github/pull/2184](https://togithub.com/integrations/terraform-provider-github/pull/2184) #### New Contributors - [@​dschniepp](https://togithub.com/dschniepp) made their first contribution in [https://github.com/integrations/terraform-provider-github/pull/2185](https://togithub.com/integrations/terraform-provider-github/pull/2185) - [@​riezebosch](https://togithub.com/riezebosch) made their first contribution in [https://github.com/integrations/terraform-provider-github/pull/2154](https://togithub.com/integrations/terraform-provider-github/pull/2154) - [@​bradam12](https://togithub.com/bradam12) made their first contribution in [https://github.com/integrations/terraform-provider-github/pull/2164](https://togithub.com/integrations/terraform-provider-github/pull/2164) - [@​ErikElkins](https://togithub.com/ErikElkins) made their first contribution in [https://github.com/integrations/terraform-provider-github/pull/2155](https://togithub.com/integrations/terraform-provider-github/pull/2155) - [@​laughedelic](https://togithub.com/laughedelic) made their first contribution in [https://github.com/integrations/terraform-provider-github/pull/2184](https://togithub.com/integrations/terraform-provider-github/pull/2184) **Full Changelog**: integrations/terraform-provider-github@v6.0.1...v6.1.0 </details> --- ### Configuration 📅 **Schedule**: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined). 🚦 **Automerge**: Disabled by config. Please merge this manually once you are satisfied. ♻ **Rebasing**: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox. 🔕 **Ignore**: Close this PR and you won't be reminded about this update again. --- - [ ] <!-- rebase-check -->If you want to rebase/retry this PR, check this box --- This PR has been generated by [Renovate Bot](https://togithub.com/renovatebot/renovate). <!--renovate-debug:eyJjcmVhdGVkSW5WZXIiOiIzNy4yNDAuMSIsInVwZGF0ZWRJblZlciI6IjM3LjI1Ny4wIiwidGFyZ2V0QnJhbmNoIjoibWFpbiJ9--> Co-authored-by: kireque-bot[bot] <143391978+kireque-bot[bot]@users.noreply.github.com>
Resolves part of #1199
This adds a new resource
github_actions_enterprise_permissions
to control GitHub Actions policy on an enterprise.Pull request checklist
Does this introduce a breaking change?
Please see our docs on breaking changes to help!