Put the new behavior behind the flag

suzuki-shunsuke · Feb 14, 2024 · 246a10d · 246a10d
1 parent 240f2ac
commit 246a10d
Show file tree

Hide file tree

Showing 12 changed files with 58 additions and 11 deletions.
diff --git a/check-terraform-skip/dist/index.js b/check-terraform-skip/dist/index.js
diff --git a/export-aws-secrets-manager/dist/index.js b/export-aws-secrets-manager/dist/index.js
diff --git a/get-global-config/action.yaml b/get-global-config/action.yaml
@@ -31,6 +31,9 @@ outputs:
   enable_trivy:
     description: If true, trivy is enabled in test action
 
+  disable_update_local_path_module_caller:
+    description: If true, local-path module caller is updated when any of local-path modules is updated
+
   aqua_update_checksum_enabled:
     description: If true, aqua-checksums.json on working directories is updated
   aqua_update_checksum_skip_push:

diff --git a/get-global-config/dist/index.js b/get-global-config/dist/index.js
diff --git a/get-global-config/src/run.ts b/get-global-config/src/run.ts
@@ -27,6 +27,7 @@ interface Outputs {
   drift_issue_repo_owner: string;
   drift_issue_repo_name: string;
   disable_update_related_pull_requests: boolean;
+  disable_update_local_path_module_caller: boolean;
   aqua_update_checksum_enabled: boolean;
   aqua_update_checksum_prune: boolean;
   aqua_update_checksum_skip_push: boolean;
@@ -64,6 +65,9 @@ export const main = (config: lib.Config, input: Input): Result => {
     disable_update_related_pull_requests: !(
       config?.update_related_pull_requests?.enabled ?? true
     ),
+    disable_update_local_path_module_caller: !(
+      config?.update_local_path_module_caller?.enabled ?? false
+    ),
     aqua_update_checksum_enabled:
       config?.aqua?.update_checksum?.enabled ?? false,
     aqua_update_checksum_prune: config?.aqua?.update_checksum?.prune ?? false,

diff --git a/get-target-config/dist/index.js b/get-target-config/dist/index.js
diff --git a/lib/dist/index.d.ts b/lib/dist/index.d.ts
diff --git a/lib/dist/index.js b/lib/dist/index.js
diff --git a/lib/src/index.ts b/lib/src/index.ts
@@ -182,6 +182,11 @@ const Config = z.object({
   tflint: z.optional(TflintConfig),
   tfsec: z.optional(TfsecConfig),
   trivy: z.optional(TrivyConfig),
+  update_local_path_module_caller: z.optional(
+    z.object({
+      enabled: z.optional(z.boolean()),
+    }),
+  ),
   update_related_pull_requests: z.optional(
     z.object({
       enabled: z.optional(z.boolean()),

diff --git a/list-targets-with-changed-files/dist/index.js b/list-targets-with-changed-files/dist/index.js
diff --git a/list-targets-with-changed-files/src/run.ts b/list-targets-with-changed-files/src/run.ts
@@ -153,7 +153,7 @@ export const run = (input: Input): TargetConfig[] => {
     }
   }
 
-  const moduleCallerMap = JSON.parse(core.getInput("module_callers"));
+  const moduleCallerMap = JSON.parse(core.getInput("module_callers") || "{}");
   const changedWorkingDirs = new Set<string>();
   for (let i = 0; i < changedFiles.length; i++) {
     const changedFile = changedFiles[i];

diff --git a/list-targets/action.yaml b/list-targets/action.yaml
@@ -43,8 +43,11 @@ runs:
       shell: bash
     - uses: suzuki-shunsuke/tfaction/list-working-dirs@main
       id: list-working-directory-configs
-    - uses: suzuki-shunsuke/list-module-callers@main
+    - uses: suzuki-shunsuke/tfaction/get-global-config@main
+      id: global-config
+    - uses: suzuki-shunsuke/tfaction/list-module-callers@main
       id: list-module-callers
+      if: steps.global-config.outputs.disable_update_local_path_module_caller == 'false'
       with:
         config_files: ${{ steps.list-working-directory-configs.outputs.file }}
         module_files: ${{ steps.list-working-directory-configs.outputs.module_file }}