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

added attributes to github_user and github_team data sources #2

Merged
merged 3 commits into from
Jun 16, 2017
Merged

added attributes to github_user and github_team data sources #2

merged 3 commits into from
Jun 16, 2017

Conversation

mootpt
Copy link

@mootpt mootpt commented Jun 12, 2017

  • updated go-github
  • added gpg and ssh keys to github_user data source and members to github_team
  • fixed issues with attributes using pointers instead of functions
  • documented changes

JWal31 added 3 commits June 12, 2017 10:16
- Added gpg_keys and ssh_key list to github_user data source
- Added members list to github_team data source
@josephholsten
Copy link

Looks great! Do we need to change things anywhere else for the vendor update?

@mootpt
Copy link
Author

mootpt commented Jun 14, 2017

@josephholsten nope. should be good to merge.

@stack72
Copy link
Contributor

stack72 commented Jun 16, 2017

Hi @mootpt

This LGTM! Thanks for this :)

P.

@stack72 stack72 merged commit d320a50 into integrations:master Jun 16, 2017
jcudit pushed a commit that referenced this pull request Mar 25, 2020
Refactor the error catching logic and be explicit about checking for a 
nil pointer.
jcudit pushed a commit that referenced this pull request Mar 23, 2021
Update fork to match upstream
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
added attributes to github_user and github_team data sources
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
Refactor the error catching logic and be explicit about checking for a 
nil pointer.
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.

3 participants