Keegan Campbell
Keegan Campbell
I've been able to see successful integration tests before and after this change. Thanks!
I've run the PR checks and gave a couple of ideas on how to change the PR. Please let me know if you'd like to pair on updating this!
Would you mind sharing a little more details on how this occurred? Did it just start happening, or has this happened to you all along? Were you able to create...
It looks like there's a variable called `githubae_org` in your output. Are you using AE? AE does not support EMUs, so using an [IdP team sync](https://registry.terraform.io/providers/integrations/github/latest/docs/resources/team_sync_group_mapping) is probably the best...
> github_team_sync_group_mapping - throws error, infact - github_emu_group_mapping is mapping correctly in GitHubAE but the terraform is throwing error to parse the results. Do you mind giving more information about...
I've updated #1185 a bit today, and hopefully will wrap this up next week.
@k24dizzle apologies for the slow response time. I'll shoot to get this into the next release!
@k24dizzle Apologies for the delay. Integration tests on this aren't passing for me: ``` --- FAIL: TestAccGithubBranchesDataSource (83.24s) --- FAIL: TestAccGithubBranchesDataSource/manages_branches_of_a_new_repository (80.94s) --- SKIP: TestAccGithubBranchesDataSource/manages_branches_of_a_new_repository/with_an_anonymous_account (0.00s) --- SKIP: TestAccGithubBranchesDataSource/manages_branches_of_a_new_repository/with_an_individual_account (0.00s)...
cc @Olin-Vorechovsky I'd rather do this than error if there's no token present, since there's a lot of valid use cases for the provider without auth configured IMO. Thoughts?
I think this is a worthwhile change and since it's not breaking backwards compatibility, I've added it to a future milestone for release.