Skip to content

fix: reverting the multiple repository feature, as it needs more thought#75

Merged
gambol99 merged 1 commit into
mainfrom
fix/remove_multiple_repositories
Nov 14, 2025
Merged

fix: reverting the multiple repository feature, as it needs more thought#75
gambol99 merged 1 commit into
mainfrom
fix/remove_multiple_repositories

Conversation

@gambol99
Copy link
Copy Markdown
Member

@gambol99 gambol99 commented Nov 14, 2025

Before we used the repository for the state key - changing to the name breaks things. Though i’d like to support multiple repositories, the current implementation is not fit for purpose - Holding my head in shame and reverting.

@gambol99 gambol99 requested a review from teodorz November 14, 2025 10:11
@github-actions
Copy link
Copy Markdown

Pull Request Review Status

  • 🖌 Terraform Format and Style: success
  • 🔍 Terraform Linting: success
  • 👮 Terraform Security Check: success
  • 👮 Terraform Security Checkov: success
  • 🔘 Terraform Tests: success
  • 🔧 Terraform Initialisation: success
  • 🤖 Terraform Validation: success
  • 🤖 Terraform Example Validation: success
  • 📖 Terraform Documentation: success
  • 🔖 Commitlint: success

Working Directory: modules/remote_state
Pusher: @gambol99, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-aws-oidc/actions/runs/19361295426

@github-actions
Copy link
Copy Markdown

Pull Request Review Status

  • 🖌 Terraform Format and Style: success
  • 🔍 Terraform Linting: success
  • 👮 Terraform Security Check: success
  • 👮 Terraform Security Checkov: success
  • 🔘 Terraform Tests: success
  • 🔧 Terraform Initialisation: success
  • 🤖 Terraform Validation: success
  • 🤖 Terraform Example Validation: success
  • 📖 Terraform Documentation: success
  • 🔖 Commitlint: success

Working Directory: modules/provider
Pusher: @gambol99, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-aws-oidc/actions/runs/19361295426

@github-actions
Copy link
Copy Markdown

Pull Request Review Status

  • 🖌 Terraform Format and Style: success
  • 🔍 Terraform Linting: success
  • 👮 Terraform Security Check: success
  • 👮 Terraform Security Checkov: success
  • 🔘 Terraform Tests: success
  • 🔧 Terraform Initialisation: success
  • 🤖 Terraform Validation: success
  • 🤖 Terraform Example Validation: success
  • 📖 Terraform Documentation: success
  • 🔖 Commitlint: success

Working Directory: modules/role
Pusher: @gambol99, Action: pull_request
Workflow Run Link: https://github.com/appvia/terraform-aws-oidc/actions/runs/19361295426

Copy link
Copy Markdown

@teodorz teodorz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@gambol99 gambol99 merged commit a0b5eff into main Nov 14, 2025
36 checks passed
@gambol99 gambol99 deleted the fix/remove_multiple_repositories branch November 14, 2025 10:48
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.

2 participants