Skip to content

fix: Correct repository_name_templates when using single repo#103

Merged
jaredfholgate merged 2 commits into
Azure:release/github-multi-repo-fixfrom
richeney:single-repo-terraform-github
Jul 11, 2025
Merged

fix: Correct repository_name_templates when using single repo#103
jaredfholgate merged 2 commits into
Azure:release/github-multi-repo-fixfrom
richeney:single-repo-terraform-github

Conversation

@richeney

@richeney richeney commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

Overview/Summary

Set the uses: value correctly when using single repo in combincation with Terraform and GitHub.

Replicates the logic used for azuredevops / terraform, and applies it to github / terraform.

This PR fixes

1 #391

Testing Evidence

Tested on bootstrap in pwsh on Ubuntu 24.04, and then ran the two workflows.

As part of this Pull Request I have

  • Checked for duplicate Pull Requests
  • Associated it with relevant issues, for tracking and closure.
  • Ensured my code/branch is up-to-date with the latest changes in the main branch
  • Performed testing and provided evidence.
  • Updated relevant and associated documentation.

@richeney richeney left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Also corrects an issue where the input.terraform_cli_version variable was referenced as input.terraform_cli_versions.

@richeney

richeney commented Jul 9, 2025

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree [company="Microsoft"]

@richeney richeney changed the title Correct repository_name_templates when using single repo Fix: Correct repository_name_templates when using single repo Jul 9, 2025

@jaredfholgate jaredfholgate left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please run terraform fmt -recursive or make fmt. Thanks

@microsoft-github-policy-service microsoft-github-policy-service Bot added the Needs: Author Feedback 👂 Needs the author to provide feedback label Jul 9, 2025
@jaredfholgate

Copy link
Copy Markdown
Member

Just closing and opening to see if the license check picks up.

@microsoft-github-policy-service microsoft-github-policy-service Bot removed the Needs: Author Feedback 👂 Needs the author to provide feedback label Jul 11, 2025
@richeney

Copy link
Copy Markdown
Contributor Author

@microsoft-github-policy-service agree

@richeney richeney changed the title Fix: Correct repository_name_templates when using single repo fix: Correct repository_name_templates when using single repo Jul 11, 2025
@jaredfholgate jaredfholgate added the PR: Safe to test 🧪 Enables running of End to End Tests label Jul 11, 2025
@jaredfholgate
jaredfholgate changed the base branch from main to release/github-multi-repo-fix July 11, 2025 13:20
@jaredfholgate
jaredfholgate merged commit 1b3cd4c into Azure:release/github-multi-repo-fix Jul 11, 2025
9 of 33 checks passed
jaredfholgate added a commit that referenced this pull request Jul 11, 2025
…104)

* Correct repository_name_templates when using single repo

* terraform fmt -recursive

---------

Co-authored-by: Richard Cheney <24895350+richeney@users.noreply.github.com>
Co-authored-by: Richard Cheney <richeney@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

PR: Safe to test 🧪 Enables running of End to End Tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants