Keegan Campbell

Results 308 comments of 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.