Skip to content

Conversation

@stevehipwell
Copy link
Collaborator

Resolves #2493


Before the change?

  • The github_branch_protection_v3 resource churned when when using checks or contexts in required_status_checks

After the change?

  • No churning on github_branch_protection_v3 resource

Pull request checklist

  • Tests for the changes have been added (for bug fixes / features)
  • Docs have been reviewed and added / updated if needed (for bug fixes / features)

Does this introduce a breaking change?

Please see our docs on breaking changes to help!

  • Yes
  • No

@alexstojda
Copy link

Also seeing this behaviour, would be great to get this merged!

@stevehipwell stevehipwell force-pushed the fix-branch-protection-churn branch from a42bada to 1eebbb1 Compare January 14, 2025 12:07
@RobinOlofsson
Copy link

Bump!

@joshmyers
Copy link

joshmyers commented Mar 26, 2025

Also hitting this. - maybe @kfcampbell can help?

@KrawczowaKris
Copy link

@kfcampbell @nickfloyd
could you take a look please? this is very important to us and we would like to see this change made

@stevehipwell stevehipwell force-pushed the fix-branch-protection-churn branch from 1eebbb1 to 066cf52 Compare June 5, 2025 14:50
@stevehipwell stevehipwell force-pushed the fix-branch-protection-churn branch from 066cf52 to 6233773 Compare June 27, 2025 08:59
@joaorpsoares
Copy link

Bump 🙏🏼

@stevehipwell
Copy link
Collaborator Author

@joaorpsoares you'd probably be better off switching over to rulesets as they (mostly) work and don't have this churn issue.

@stevehipwell stevehipwell force-pushed the fix-branch-protection-churn branch from 6233773 to 7e8a8be Compare October 20, 2025 17:25
@stevehipwell
Copy link
Collaborator Author

@nickfloyd this is a real pain point, albeit for a resource with a newer replacement. I could add a test to go with this, but I likely won't have time until the end of the week.

@nickfloyd nickfloyd moved this from Backlog to In Progress in Terraform Provider Oct 21, 2025
@nickfloyd nickfloyd mentioned this pull request Oct 21, 2025
4 tasks
@nickfloyd
Copy link
Contributor

@stevehipwell Tests have been added here #2799

@nickfloyd nickfloyd merged commit 67382ff into integrations:main Oct 21, 2025
3 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in 🧰 Octokit Active Oct 21, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Terraform Provider Oct 21, 2025
@stevehipwell stevehipwell deleted the fix-branch-protection-churn branch October 22, 2025 09:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

[BUG]: The github_branch_protection_v3 resource churns when using checks or contexts in required_status_checks

7 participants