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

New Resource: github_repository_project #115

Merged
merged 6 commits into from
Aug 10, 2018

Conversation

shihanng
Copy link
Contributor

@shihanng shihanng commented Aug 8, 2018

Related to #111, this allow us to create projects on repository level via Terraform.
https://developer.github.com/v3/projects/#create-a-repository-project

I believe this fixes #19.

Copy link
Contributor

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

Thanks for this PR too @shihanng

Without repeating myself here I'm just going to refer to https://github.com/terraform-providers/terraform-provider-github/pull/111 as all my comments/questions apply here as well, except the one about docs.

@shihanng shihanng force-pushed the repository_project branch from 3df58fe to dd35451 Compare August 9, 2018 15:08
@shihanng
Copy link
Contributor Author

shihanng commented Aug 9, 2018

@radeksimko, I've made changes based on your comments:

  • inline struct fields for readability
  • add test for import and fix import based on the issue discovered by the unit-test
  • rebased on master

Could you please have another look? Thank you.

Copy link
Contributor

@radeksimko radeksimko left a comment

Choose a reason for hiding this comment

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

Thanks @shihanng, I just made a few small tweaks (commit messages should be self-explanatory) and pushed these straight to your branch, so we can pull this PR in. I hope you don't mind.

🚢

@radeksimko radeksimko merged commit 16f59f9 into integrations:master Aug 10, 2018
@shihanng
Copy link
Contributor Author

Thank you 🎉

@shihanng shihanng deleted the repository_project branch August 10, 2018 05:39
kfcampbell pushed a commit to kfcampbell/terraform-provider-github that referenced this pull request Jul 26, 2022
* r/github_repository_project: first commit

* r/github_repository_project: add docs

* inline struct fields for readability

* repository_project: fix import and add test

* Clean up var names & fix typo

* Improve error messages
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FEATURE REQUEST] Add Support for Repository Projects
2 participants