Merge pull request #57 from silinternational/develop

run deploy workflow on branches, not tags, not Terraform
silinternational · Nov 6, 2024 · 83e9c91 · 83e9c91
2 parents 8573f19 + ab4e850
commit 83e9c91
Show file tree

Hide file tree

Showing 5 changed files with 25 additions and 20 deletions.
diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS
@@ -0,0 +1,4 @@
+* @silinternational/developers
+/terraform @silinternational/tf-devs
+*.go @silinternational/go-devs
+go.* @silinternational/go-devs
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -2,6 +2,8 @@ name: Continuous Integration
 
 on:
   push:
+    branches: [ '**' ]
+    paths-ignore: [ 'terraform/**' ]
 
 jobs:
   test-and-deploy:

diff --git a/terraform/.terraform.lock.hcl b/terraform/.terraform.lock.hcl
diff --git a/terraform/main.tf b/terraform/main.tf
@@ -4,8 +4,7 @@
  */
 module "serverless-user" {
   source  = "silinternational/serverless-user/aws"
-  version = "0.1.3"
+  version = "~> 0.4"
 
   app_name   = "app-monitoring-archiver"
-  aws_region = var.aws_region
 }
diff --git a/terraform/versions.tf b/terraform/versions.tf
@@ -4,7 +4,7 @@ terraform {
   required_providers {
     aws = {
       source  = "hashicorp/aws"
-      version = "~> 4.0"
+      version = "~> 5.0"
     }
   }
 }
-Original file line number
+Diff line change
@@ Expand Up / @@ -2,6 +2,8 @@ name: Continuous Integration @@
     on:
       push:
+        branches: [ '**' ]
+        paths-ignore: [ 'terraform/**' ]
     jobs:
       test-and-deploy:
@@ Expand Down @@