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 support for github actions variables #1569

Merged
merged 6 commits into from
Mar 10, 2023
Merged

Conversation

tjcorr
Copy link
Contributor

@tjcorr tjcorr commented Feb 25, 2023

Resolves #1534


Behavior

After the change?

Adds 3 new resources to support GitHub Actions variables at the repo, environment, and organization levels:

  • github_actions_variable
  • github_actions_environment_variable
  • github_actions_organization_variable

In addition this also adds the corresponding data sources:

  • github_actions_variables
  • github_actions_environment_variables
  • github_actions_organization_variables

As well as the missing data source github_actions_environment_secrets

Other information

This required an update to a new version of the github.com/google/go-github/v50 module (now v50.1.0) to support the new APIs.


Additional info

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)
  • Added the appropriate label for the given change

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes (Please add the Type: Breaking change label)
  • No

Pull request type

Please add the corresponding label for change this PR introduces:

  • Bugfix: Type: Bug
  • Feature/model/API additions: Type: Feature
  • Updates to docs or samples: Type: Documentation
  • Dependencies/code cleanup: Type: Maintenance

@tjcorr
Copy link
Contributor Author

tjcorr commented Feb 25, 2023

I wasn't sure if there was a process to auto-generate the docs for the new resources/data sources. If not just let me know and I can try and write some.

@ilmax
Copy link
Contributor

ilmax commented Feb 27, 2023

I wasn't sure if there was a process to auto-generate the docs for the new resources/data sources. If not just let me know and I can try and write some.

Hello @tjcorr, I asked the same a while ago here

@tjcorr
Copy link
Contributor Author

tjcorr commented Feb 28, 2023

I added in support for importing existing variables as well as the new docs pages. Should be ready for a review at this point.

Copy link
Member

@kfcampbell kfcampbell left a comment

Choose a reason for hiding this comment

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

This looks really good, thank you @tjcorr! If you don't want to/don't have time to link the docs, let me know and I can take care of it before getting this in.

@yacinehmito
Copy link

Thank you very much @tjcorr! Can't wait for the release.

Copy link
Member

@kfcampbell kfcampbell left a comment

Choose a reason for hiding this comment

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

@tjcorr You're the best, thank you!

@kfcampbell kfcampbell merged commit 62d2644 into integrations:main Mar 10, 2023
reedloden pushed a commit to reedloden/terraform-provider-github that referenced this pull request Jun 14, 2023
* add support for github actions variables

* adding importers and website documentation

* Run go mod tidy

* update website nav with new pages

---------

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
bazaah added a commit to amachsoftware/terraform-github-repository that referenced this pull request Aug 9, 2023
From v4.20, to support the usage of the action_variable resource
(included in v5.18.3)

References: https://github.com/integrations/terraform-provider-github/releases/tag/v5.18.3
References: integrations/terraform-provider-github#1569
bazaah added a commit to amachsoftware/terraform-github-repository that referenced this pull request Aug 9, 2023
From v4.20, to support the usage of the action_variable resource
(included in v5.18.3)

References: https://github.com/integrations/terraform-provider-github/releases/tag/v5.18.3
References: integrations/terraform-provider-github#1569
avidspartan1 pushed a commit to avidspartan1/terraform-provider-github that referenced this pull request Feb 5, 2024
* add support for github actions variables

* adding importers and website documentation

* Run go mod tidy

* update website nav with new pages

---------

Co-authored-by: Keegan Campbell <me@kfcampbell.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEAT]: Environment Variables for Github Actions
4 participants