Skip to content

ci: bump framework_ref to provider 6.12.1 test#41

Merged
NWarila merged 1 commit into
mainfrom
fix/bump-framework-provider-6.12
May 26, 2026
Merged

ci: bump framework_ref to provider 6.12.1 test#41
NWarila merged 1 commit into
mainfrom
fix/bump-framework-provider-6.12

Conversation

@NWarila
Copy link
Copy Markdown
Owner

@NWarila NWarila commented May 26, 2026

Tests whether terraform-provider-github 6.12.1 correctly omits allow_forks from PATCH payloads when Terraform value is null. See nwarila-platform/github-terraform-framework#63.

Before After
uses: SHA abda8e1f 7f2cf0b6
framework_ref abda8e1f 7f2cf0b6

If deploy succeeds for the 3 personal-account private repos after this lands, F-A is resolved.

Bumps framework pin to the integrations/github provider 6.12.1 fix
landed in nwarila-platform/github-terraform-framework#63.

Tests whether the newer provider correctly omits `allow_forks` from
PATCH payloads when the Terraform value is null. If yes, the
allow_forking ownership-aware default from #62 finally takes effect
and personal-account private repos (Resume, Personal, github-sandbox)
deploy cleanly. If no, the fallback is the `managed: optional(bool,
true)` schema flag (separate framework PR).

| | Before | After |
|---|---|---|
| uses: SHA | abda8e1f | 7f2cf0b6 |
| framework_ref | abda8e1f | 7f2cf0b6 |

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@NWarila NWarila merged commit 8da00c8 into main May 26, 2026
20 checks passed
@NWarila NWarila deleted the fix/bump-framework-provider-6.12 branch May 26, 2026 14:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant