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.

Sign up for GitHub

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

Jump to bottom

Update to 6.2.1 from upstream #14

Merged
merged 49 commits into from
Jun 12, 2024
Merged

Update to 6.2.1 from upstream #14

merged 49 commits into from
Jun 12, 2024

Conversation

snovikov
Copy link

Resolves #ISSUE_NUMBER

Before the change?

After the change?

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

rwblokzijl and others added 30 commits January 19, 2024 10:33
@rwblokzijl @kfcampbell
Make resource repository_deploy_key read_only a bool, and expand the …
47c3d36 
…example (integrations#2044)

* Make resource repository_deploy_key read_only a bool, and expand the example

* Clarify used provider

---------

Co-authored-by: Keegan Campbell <[email protected]>
@Nmishin @kfcampbell
Add prevent_self_review parameter to the github_repository_environmen…
dbda378 
…t resource (integrations#2000)

Co-authored-by: Keegan Campbell <[email protected]>
@srgustafson8 @kfcampbell
fix: github_rest_api data source always returns header and body as nu…
6e507a6 
…ll (integrations#2110)

* Enhance tests

* Marshal body and headers to JSON string

* Update documentation to reference JSON string

---------

Co-authored-by: Keegan Campbell <[email protected]>
@dependabot
build(deps): bump github.com/google/uuid from 1.5.0 to 1.6.0
8e61a74 
Bumps [github.com/google/uuid](https://github.com/google/uuid) from 1.5.0 to 1.6.0.
- [Release notes](https://github.com/google/uuid/releases)
- [Changelog](https://github.com/google/uuid/blob/master/CHANGELOG.md)
- [Commits](google/uuid@v1.5.0...v1.6.0)

---
updated-dependencies:
- dependency-name: github.com/google/uuid
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot
build(deps): bump peter-evans/create-or-update-comment from 3 to 4
86b20f2 
Bumps [peter-evans/create-or-update-comment](https://github.com/peter-evans/create-or-update-comment) from 3 to 4.
- [Release notes](https://github.com/peter-evans/create-or-update-comment/releases)
- [Commits](peter-evans/create-or-update-comment@v3...v4)

---
updated-dependencies:
- dependency-name: peter-evans/create-or-update-comment
  dependency-type: direct:production
  update-type: version-update:semver-major
...

Signed-off-by: dependabot[bot] <[email protected]>
@nickfloyd
build(deps): bump peter-evans/create-or-update-comment from 3 to 4
378d451
@nickfloyd
build(deps): bump github.com/google/uuid from 1.5.0 to 1.6.0
ddb0c3b
@Nmishin @kfcampbell
Add support for packages IP ranges in github_ip_ranges (integrations#…
970dd20 
…1958)

* Add support for packages IP ranges in github_ip_ranges

* Update docks for packages IP ranges in github_ip_ranges

* gofmt

* Add tests for packages IP ranges in github_ip_ranges

* Update tests for packages IP ranges in github_ip_ranges

---------

Co-authored-by: Keegan Campbell <[email protected]>
@arunsathiya @kfcampbell
fix(resource_github_release): Handle missing release in read (integra…
9008816 
…tions#2115)

Co-authored-by: Keegan Campbell <[email protected]>
@manjinder-mckc @kfcampbell
docs: update commit_message arg desc for repository_file resource (in…
414299f 
…tegrations#2125)

Co-authored-by: Keegan Campbell <[email protected]>
@bpaquet
fix doc for integrations#2054: Clarify how allows_force_pushes is wor…
b4235f7 
…king
@bpaquet
Improve doc
ee40a30
@bpaquet
Fix
5de1f37
@bpaquet
Fix
7be79c4
@dependabot
build(deps): bump golang.org/x/oauth2 from 0.16.0 to 0.17.0
b28bf62 
Bumps [golang.org/x/oauth2](https://github.com/golang/oauth2) from 0.16.0 to 0.17.0.
- [Commits](golang/oauth2@v0.16.0...v0.17.0)

---
updated-dependencies:
- dependency-name: golang.org/x/oauth2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@nickfloyd
build(deps): bump golang.org/x/oauth2 from 0.16.0 to 0.17.0
684398f
@grahamhar @kfcampbell
docs: Correct name of docs file (integrations#2138)
3eb0fbd 
Co-authored-by: Keegan Campbell <[email protected]>
@nickfloyd
docs: Clarify how allows_force_pushes works
2d31318 
fix doc for integrations#2054: Clarify how allows_force_pushes is working
@kfcampbell @dcfranca
@nickfloyd @felixlut @georgekaz @richjyoung
v6 (integrations#2091)
16872b7 
* Add retryable transport for errors (integrations#1704)

* Add retryable transport for errors

In order to address the issue integrations#210
I have added 3 new parameters to the provider

- retry_delay_ms
- max_retries
- retryable_errors

In case max_retries is > 0 (defaults to zero)
it will retry the request in case it is an error
the retryable_errors defaults to [500, 502, 503, 504]

It retries after the ms specified in retry_delay_ms (defaults to 1000)

* Update documentation with new parameters for retry

* Change default of max_retries to 3

* Fix typo in naming

* Update github/transport_test.go

* Add error check for data seek

* Consolidate the default retriable errors on a function

* Fix typo on the comments

Co-authored-by: Keegan Campbell <[email protected]>

* Update vendor

* Fix merging conflicts

* Update documentation with new parameters for retry

* Change default of max_retries to 3

* Fix typo in naming

* Add error check for data seek

* Update github/transport_test.go

* Consolidate the default retriable errors on a function

* Fix typo on the comments

Co-authored-by: Keegan Campbell <[email protected]>

* Don't run go mod tidy on release (integrations#1788)

* Don't run go mod tidy on release

* Be more specific about releases

* Fix lint with APIMeta deprecation

---------

Co-authored-by: Keegan Campbell <[email protected]>
Co-authored-by: Nick Floyd <[email protected]>

* fix: remove repository topic from state if it doesnt exist in GitHub anymore (integrations#1918)

* remove repository topic if they cannot be found in GitHub anymore

* Fix build error by bumping package version in offending file

---------

Co-authored-by: Keegan Campbell <[email protected]>

* Bump version to v6 (integrations#2106)

* Upgrade to Terraform Plugin SDK v2 (integrations#1780)

* go mod tidy -go=1.16 && go mod tidy -go=1.17

* Run go mod vendor

* Attempt v2 upgrade

* Plugin compiling

* Fix some provider test errors

* Fix test compilation error

* ValidateFunc --> ValidateDiagFunc

* Fix casing

* Sprinkle toDiagFunc everywhere

* More fixes for validation functions

* State --> StateContext

* %s --> %v when printing diags

* ConfigureFunc --> ConfigureContextFunc

* Checking results of d.Set, round one

* Continue checking d.Set results

* Check results of d.Set, round three

* Checking d.Set results, round four

* d.Set round five

* In tests, export GITHUB_TEST_ORGANIZATION

* Remove unnecessary MaxItems on computed value

* Go build now works

* Resolve linting errors

* Apply diag.FromErr twice more

* Pass key names into toDiagFunc helper

* Construct cty.Path from strings

* Tests now working

* Update terraform-plugin-sdk version

* Remove commented attribute setting in resource_github_team.go

* Fix restrict pushes on github_branch_protection. Fix branch protection tests (integrations#2045)

* Update restrict pushes. Fix branch protection tests

Change blocks_creations default to true. Fix broken build.

* add state migration

* rename push_restrictions to push_allowances

* correct state migration issue

* add docs clarification

* update migration func args

* fix test args

* cleanup tests

* Pass context.Background() in test

* fix timestamp fields

---------

Co-authored-by: Keegan Campbell <[email protected]>

* Set group_id correctly (integrations#2133)

* Run go get -u github.com/golangci/golangci-lint

* Separate github_team_members import from github_team as create_default_maintainers is not defined for members resource (integrations#2126)

Co-authored-by: Keegan Campbell <[email protected]>

* Add missing variable definition for test case

---------

Co-authored-by: Daniel França <[email protected]>
Co-authored-by: Nick Floyd <[email protected]>
Co-authored-by: Felix Luthman <[email protected]>
Co-authored-by: georgekaz <[email protected]>
Co-authored-by: Rich Young <[email protected]>
@dependabot
build(deps): bump github.com/golangci/golangci-lint (integrations#2159)
bf5403d 
Bumps [github.com/golangci/golangci-lint](https://github.com/golangci/golangci-lint) from 1.56.1 to 1.56.2.
- [Release notes](https://github.com/golangci/golangci-lint/releases)
- [Changelog](https://github.com/golangci/golangci-lint/blob/master/CHANGELOG.md)
- [Commits](golangci/golangci-lint@v1.56.1...v1.56.2)

---
updated-dependencies:
- dependency-name: github.com/golangci/golangci-lint
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot @kfcampbell
build(deps): bump github.com/hashicorp/terraform-plugin-sdk/v2 (integ…
ceadb68 
…rations#2160)

Bumps [github.com/hashicorp/terraform-plugin-sdk/v2](https://github.com/hashicorp/terraform-plugin-sdk) from 2.31.0 to 2.32.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](hashicorp/terraform-plugin-sdk@v2.31.0...v2.32.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-sdk/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Keegan Campbell <[email protected]>
@dependabot
build(deps): bump github.com/hashicorp/terraform-plugin-sdk/v2
5d790cb 
Bumps [github.com/hashicorp/terraform-plugin-sdk/v2](https://github.com/hashicorp/terraform-plugin-sdk) from 2.32.0 to 2.33.0.
- [Release notes](https://github.com/hashicorp/terraform-plugin-sdk/releases)
- [Changelog](https://github.com/hashicorp/terraform-plugin-sdk/blob/main/CHANGELOG.md)
- [Commits](hashicorp/terraform-plugin-sdk@v2.32.0...v2.33.0)

---
updated-dependencies:
- dependency-name: github.com/hashicorp/terraform-plugin-sdk/v2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@rnestler
Update example usage to use version 6.0
d552d76
@tomasmota
Fix github_external_groups page title
ba93d48
@nickfloyd
build(deps): bump github.com/hashicorp/terraform-plugin-sdk/v2 from 2…
ca7938e 
….32.0 to 2.33.0
@nickfloyd
Merge branch 'main' into main
e7b3083
@nickfloyd
docs: Fix github_external_groups page title
7e0d468
@nickfloyd
docs: Update example usage to use version 6.0
3f27927 
docs: Update example usage to use version 6.0
@Danielku15 @kfcampbell
fix: Use desired state for determining actions config (integrations#2114
97d5113 
)

Co-authored-by: Keegan Campbell <[email protected]>
@felixlut @kfcampbell
GitHub org ignore archived repos (integrations#1833)
4ca9b21 
* add ignore_archived_repos argument to github_organization datasource

* add test for checking that the archived repo is ignored

* add documentation for ignore_archived_repos

* fix ordering of imports

---------

Co-authored-by: Keegan Campbell <[email protected]>
dependabot bot and others added 19 commits March 4, 2024 10:44
@dependabot @kfcampbell
build(deps): bump github.com/stretchr/testify from 1.8.4 to 1.9.0 (in…
3859526 
…tegrations#2177)

Bumps [github.com/stretchr/testify](https://github.com/stretchr/testify) from 1.8.4 to 1.9.0.
- [Release notes](https://github.com/stretchr/testify/releases)
- [Commits](stretchr/testify@v1.8.4...v1.9.0)

---
updated-dependencies:
- dependency-name: github.com/stretchr/testify
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Keegan Campbell <[email protected]>
@dependabot
build(deps): bump golang.org/x/crypto from 0.19.0 to 0.21.0 (integrat…
4eabe92 
…ions#2180)

Bumps [golang.org/x/crypto](https://github.com/golang/crypto) from 0.19.0 to 0.21.0.
- [Commits](golang/crypto@v0.19.0...v0.21.0)

---
updated-dependencies:
- dependency-name: golang.org/x/crypto
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot @kfcampbell
build(deps): bump actions/add-to-project from 0.5.0 to 0.6.0 (integra…
b9af9f3 
…tions#2175)

Bumps [actions/add-to-project](https://github.com/actions/add-to-project) from 0.5.0 to 0.6.0.
- [Release notes](https://github.com/actions/add-to-project/releases)
- [Commits](actions/add-to-project@v0.5.0...v0.6.0)

---
updated-dependencies:
- dependency-name: actions/add-to-project
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: Keegan Campbell <[email protected]>
@dschniepp
fix: validation to allow 1000 results per page (integrations#2185)
a69c6f0
@Danielku15 @kfcampbell
fix: Prevent loading of allowed actions if not configured (integratio…
480825c 
…ns#2186)

Co-authored-by: Keegan Campbell <[email protected]>
@riezebosch @kfcampbell
fix(data_source_github_rest_api): only allow for 404 on err (integrat…
39c5e06 
…ions#2154)

Co-authored-by: Keegan Campbell <[email protected]>
@bradam12 @nickfloyd @kfcampbell
fix: error if autolink reference not found (integrations#2164)
4cebab8 
* fix: error if autolink reference not found

* Add test for importing an invalid key

---------

Co-authored-by: Nick Floyd <[email protected]>
Co-authored-by: Keegan Campbell <[email protected]>
@ErikElkins
feat: Add github_actions_enterprise_permissions (integrations#2155)
52e1853 
* Adding github_actions_enterprise_permissions (#1)

* Adding enterprise permissions resource

* Working version

* Adding docs

* Functioning tests

* Adding check on integration tests

* Switching to github_enterprise_actions_permissions
@laughedelic @kfcampbell
docs: configure release notes categories based on labels (integration…
dd57a50 
…s#2184)

* docs: customize release notes categories based on labels

* Update release.yml

* Move dependencies under maintenance

---------

Co-authored-by: Keegan Campbell <[email protected]>
@dependabot
build(deps): bump golang.org/x/oauth2 from 0.17.0 to 0.18.0 (integrat…
30e6176 
…ions#2189)

Bumps [golang.org/x/oauth2](https://github.com/golang/oauth2) from 0.17.0 to 0.18.0.
- [Commits](golang/oauth2@v0.17.0...v0.18.0)

---
updated-dependencies:
- dependency-name: golang.org/x/oauth2
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@riezebosch @kfcampbell
fix(data_source_github_rest_api): store raw body in state (integratio…
af693fd 
…ns#2152)

Co-authored-by: Keegan Campbell <[email protected]>
@dependabot
build(deps): bump google.golang.org/protobuf from 1.32.0 to 1.33.0 (i…
5e090bc 
…ntegrations#2194)

Bumps google.golang.org/protobuf from 1.32.0 to 1.33.0.

---
updated-dependencies:
- dependency-name: google.golang.org/protobuf
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@bradam12 @kfcampbell
feat: add github_enterprise_actions_runner_group (integrations#2158)
b1243e4 
Co-authored-by: Keegan Campbell <[email protected]>
@georgekaz @matthewcostatide @kfcampbell
Feat: Fixes abandoned PR integrations#2017: Add support for the requi…
84e8d55 
…re_last_push_approval flag in github_branch_protection_v3 (integrations#2199)

* Add the require_last_push_approval flag as an option for the required_pull_request_reviews of github_branch_protection_v3

* Update github/resource_github_branch_protection_v3.go

* Update website/docs/r/branch_protection_v3.html.markdown

* Use local var to capture RequireLastPushApproval value, and pass it to the containing struct as a reference

* Add missing property read

---------

Co-authored-by: Matthew Costa <[email protected]>
Co-authored-by: Keegan Campbell <[email protected]>
Co-authored-by: Matthew Costa <[email protected]>
@dependabot
build(deps): bump actions/add-to-project from 0.6.0 to 0.6.1 (integra…
4c8424f 
…tions#2210)

Bumps [actions/add-to-project](https://github.com/actions/add-to-project) from 0.6.0 to 0.6.1.
- [Release notes](https://github.com/actions/add-to-project/releases)
- [Commits](actions/add-to-project@v0.6.0...v0.6.1)

---
updated-dependencies:
- dependency-name: actions/add-to-project
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
@dependabot
build(deps): bump github.com/golangci/golangci-lint (integrations#2209)
0ff85ac
@EttoreFoti @kfcampbell
chore(actions_runner_group): fix missing schema id (integrations#2195)
e3d487b 
Co-authored-by: Keegan Campbell <[email protected]>
@EttoreFoti @kfcampbell
chore(organization_webhook): fix insecure_ssl (integrations#2196)
e1f5feb 
Co-authored-by: Keegan Campbell <[email protected]>
@snovikov
Merge tag 'refs/tags/v5.45.0-fl.1' into update-6.2.1
49d1f2e 
# Conflicts:
#	.github/workflows/add_to_octokit_project.yml
@snovikov snovikov requested a review from a team June 12, 2024 15:04
@snovikov snovikov self-assigned this Jun 12, 2024
@github-actions GitHub Actions
Copy link

👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday! We have a process in place for prioritizing and responding to your input. Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with Status: Up for grabs. You & others like you are the reason all of this works! So thank you & happy coding! 🚀

@snovikov snovikov merged commit 09831f5 into main Jun 12, 2024
4 checks passed
@snovikov snovikov deleted the update-6.2.1 branch June 12, 2024 15:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@cytopia cytopia cytopia approved these changes

Assignees

@snovikov snovikov

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.