Skip to content

Commit bb4af09

Browse files
Merge pull request #3 from florenciacomuzzi/fix-push
fix push
2 parents 66012f4 + 2dd774b commit bb4af09

File tree

2 files changed

+7
-3
lines changed

2 files changed

+7
-3
lines changed

.github/workflows/terraform.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ env:
2626

2727
jobs:
2828
terraform:
29-
name: "Terraform Infrastructure Change Management"
29+
name: "Terraform CICD"
3030
runs-on: ubuntu-latest
3131
permissions:
3232
pull-requests: write # Required for creating or updating PRs
@@ -72,9 +72,9 @@ jobs:
7272
id: init-push
7373
if: github.event_name == 'push'
7474
run: |
75-
if [ "${{ github.ref }}" == "develop" ]; then
75+
if [ "${{ github.ref }}" == "refs/heads/develop" ]; then
7676
export ENV="dev";
77-
elif [ "${{ github.ref }}" == "main" ]; then
77+
elif [ "${{ github.ref }}" == "refs/heads/main" ]; then
7878
export ENV="prod";
7979
else
8080
echo "unsupported environment";

TODO.md

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
# TODO
2+
- use Workload Identity Federation in CICD
3+
- consider breaking up pull request and push workflows
4+
- document branching strategy

0 commit comments

Comments
 (0)