-
Notifications
You must be signed in to change notification settings - Fork 7
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
Efficient labels #105
base: master
Are you sure you want to change the base?
Efficient labels #105
Conversation
I'm putting this back in draft. The changes in the provider need more work. I'll mark it as ready for review after successful testing in pl-strflt org. Things to look out for:
|
And it's ready! You can have a look at all the labels being tracked through https://github.com/pl-strflt/github-mgmt/blob/master/github/pl-strflt.yml We might want to wait a couple of days to see if integrations/terraform-provider-github#1694 makes it into the next gh provider release since they've been very responsive lately. |
Implement issue labels support efficiently. With integrations/terraform-provider-github@main...galargh:terraform-provider-github:feat/github_issue_labels in place, we can manage all labels for a repository using a single tf resource instead of a resource per label. This greaaaaaaatly reduces the number of GitHub API calls that we'll be doing during plans/syncs/applies.
Testing