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 ListOptions to ListSelectedReposForOrgSecret #1883

Merged
merged 1 commit into from
Jun 7, 2021

Conversation

gmlewis
Copy link
Collaborator

@gmlewis gmlewis commented Jun 5, 2021

Fixes: #1882.

@gmlewis gmlewis added the Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). label Jun 5, 2021
@google-cla google-cla bot added the cla: yes Indication that the PR author has signed a Google Contributor License Agreement. label Jun 5, 2021
@codecov
Copy link

codecov bot commented Jun 5, 2021

Codecov Report

Merging #1883 (47afd73) into master (8751f3d) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #1883   +/-   ##
=======================================
  Coverage   97.65%   97.65%           
=======================================
  Files         105      105           
  Lines        6783     6786    +3     
=======================================
+ Hits         6624     6627    +3     
  Misses         86       86           
  Partials       73       73           
Impacted Files Coverage Δ
github/actions_secrets.go 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8751f3d...47afd73. Read the comment docs.

Copy link
Collaborator

@wesleimp wesleimp left a comment

Choose a reason for hiding this comment

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

LGTM 💙

@gmlewis
Copy link
Collaborator Author

gmlewis commented Jun 7, 2021

Thank you, @wesleimp !
Merging.

@gmlewis gmlewis merged commit edaee9a into google:master Jun 7, 2021
@gmlewis gmlewis deleted the i1882-add-list-options branch June 7, 2021 19:51
@patrickmarabeas
Copy link
Contributor

@gmlewis can we get a fresh release please?

@gmlewis
Copy link
Collaborator Author

gmlewis commented Jun 26, 2021

@gmlewis can we get a fresh release please?

Sure, @patrickmarabeas - feel free to LGTM/Approve #1921 to move the process along.

@gmlewis
Copy link
Collaborator Author

gmlewis commented Jun 26, 2021

https://github.com/google/go-github/releases/tag/v36.0.0 has been released.

xun-guo-anzx added a commit to xun-guo-anzx/terraform-provider-github that referenced this pull request Jun 30, 2021
* Update go-github to 36.0.0

go-github released the Go client library to [36.0.0][1] 3 days ago.

API changes includes:
* Add ListOptions required by ListSelectedReposForOrgSecret method. As described in change[2]
* Convert repoID used in CreateOrUpdateEnvSecret, GetEnvSecret and DeleteEnvSecret to int from int64. Not sure why it doesn't pop up an error before but seems like this is not a recent change in the new version

[1]: https://github.com/google/go-github/releases/tag/v36.0.0
[2]: google/go-github#1883
jcudit pushed a commit to integrations/terraform-provider-github that referenced this pull request Jul 12, 2021
* Update go-github version to 36.0.0

* Update go-github to 36.0.0

go-github released the Go client library to [36.0.0][1] 3 days ago.

API changes includes:
* Add ListOptions required by ListSelectedReposForOrgSecret method. As described in change[2]
* Convert repoID used in CreateOrUpdateEnvSecret, GetEnvSecret and DeleteEnvSecret to int from int64. Not sure why it doesn't pop up an error before but seems like this is not a recent change in the new version

[1]: https://github.com/google/go-github/releases/tag/v36.0.0
[2]: google/go-github#1883

* Fetch full list of selected repos in github_actions_organization_secret

Fix #793
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
* Update go-github version to 36.0.0

* Update go-github to 36.0.0

go-github released the Go client library to [36.0.0][1] 3 days ago.

API changes includes:
* Add ListOptions required by ListSelectedReposForOrgSecret method. As described in change[2]
* Convert repoID used in CreateOrUpdateEnvSecret, GetEnvSecret and DeleteEnvSecret to int from int64. Not sure why it doesn't pop up an error before but seems like this is not a recent change in the new version

[1]: https://github.com/google/go-github/releases/tag/v36.0.0
[2]: google/go-github#1883

* Fetch full list of selected repos in github_actions_organization_secret

Fix integrations#793
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Breaking API Change PR will require a bump to the major version num in next release. Look here to see the change(s). 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.

Actions Secrets do not support ListOptions
3 participants