Keegan Campbell

Hmm...I'm uncomfortable making this change with such a broad impact. Can it be scoped down to the one request necessary?

@riezebosch thanks for the attention here! The added test looks good to me. I'd be comfortable with adding an error check here; I think the user experience would be better...

@gr2m before I edit the tests, are you okay with the direction here? It's worked to create PRs in another repo: https://github.com/octokit/go-sdk/pull/6 Our use case here is we have a...

@gr2m I believe these test errors are due to my fork's GITHUB_TOKEN not having access to your repository. See [these partially successful tests](https://github.com/kfcampbell/create-or-update-pull-request-action/actions/runs/6909278329/job/18800294889) on my own fork. Do you have...

I've done some more research on the test cases here, and I'm pretty confused. ### First, on this PR, the tests deterministically fail: Each test fails due to a 403...

This PR (or #595, they're the same content) is now passing all expected tests and is ready for review!

I'm not sure how I would go about troubleshooting that at the provider level. I wonder if that's something specific to the Terraform platform or tooling.

@jeremyhayes I love the idea; we've unfortunately been rather resource-constrained recently and this hasn't been picked back up yet. PRs are appreciated!

@enchorb the issue being discussed here is almost always indicative of an authentication issue: either auth is not passed to the Terraform module correctly (using env vars with a PAT...

@maulik13 That is excellent information. Would you be receptive to creating a PR for our docs to describe this behavior? Ideally, we'd make this more consistent across the provider, but...