Skip to content

feat(account): make repository_prefix and repository_provider optional#326

Merged
agustincelentano merged 1 commit into
mainfrom
feat/account-optional-repository-fields
May 5, 2026
Merged

feat(account): make repository_prefix and repository_provider optional#326
agustincelentano merged 1 commit into
mainfrom
feat/account-optional-repository-fields

Conversation

@agustincelentano

Copy link
Copy Markdown
Collaborator

Summary

  • Drop module-level defaults for repository_prefix and repository_provider in the nullplatform/account module so they pass through as null when unset.
  • Aligns with terraform-provider-nullplatform#118 (released in v0.0.86), which made these fields optional on the nullplatform_account resource.
  • The provider version constraint in providers.tf is intentionally left at ~> 0.0.63 — the bump is being handled in a separate PR.

Changes

  • variables.tf: repository_prefix and repository_provider are now optional(string) (no default). slug keeps its default.
  • tests/account.tftest.hcl: uses_default_values now asserts null for the two repository fields.
  • README.md: features, architecture and inputs table updated to reflect the new behavior.

Test plan

  • tofu test passes locally (3/3)
  • pre-commit run passes on all changed files (fmt, validate, test, trailing-whitespace, detect-private-key)
  • CI green

The terraform-provider-nullplatform v0.0.86 made repository_prefix and
repository_provider optional on the nullplatform_account resource. Drop
the module-level defaults so unset values pass through as null and let
the provider treat them as unmanaged.
@agustincelentano agustincelentano force-pushed the feat/account-optional-repository-fields branch from cd32289 to aad95cf Compare May 5, 2026 17:23
@agustincelentano agustincelentano merged commit a0a079a into main May 5, 2026
42 checks passed
@agustincelentano agustincelentano deleted the feat/account-optional-repository-fields branch May 5, 2026 18:45
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.

3 participants