Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Target module callers when local path module is updated #1528

Merged
merged 21 commits into from
Mar 16, 2024
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
21 commits
Select commit Hold shift + click to select a range
240f2ac
Target module callers when module is changed
exoego Feb 10, 2024
246a10d
Put the new behavior behind the flag
exoego Feb 14, 2024
ee23e6f
Don't install when disabled
exoego Feb 17, 2024
a7d4635
Pin to the stable version
exoego Feb 17, 2024
f6d9d8b
Rename to update_local_path_module_caller and disable by default
exoego Feb 17, 2024
e150731
Call getInput in main
exoego Feb 17, 2024
1aba475
Declare Golang dependency
exoego Feb 17, 2024
3acf082
Detect go bin path
exoego Feb 17, 2024
922976c
Move to the right place since it depends on global-config
exoego Feb 17, 2024
5ab2fad
last slash not needed
exoego Feb 17, 2024
c5edbf9
style: format by prettier
suzuki-shunsuke Feb 17, 2024
6673ec7
Merge remote-tracking branch 'origin/main' into update-module-use-site
suzuki-shunsuke Feb 28, 2024
1b02f65
Merge branch 'main' into update-module-use-site
suzuki-shunsuke Mar 14, 2024
0a33035
Merge branch 'main' into update-module-use-site
suzuki-shunsuke Mar 16, 2024
8d83b3e
fix(list-module-caller): specify the install path of terraform-config…
suzuki-shunsuke Mar 16, 2024
ddf2cdd
fix(list-targets): fix typo
suzuki-shunsuke Mar 16, 2024
f0c38d5
fix(list-targets): change the install path of terraform-config-inspec…
suzuki-shunsuke Mar 16, 2024
5aea5b6
fix: stop installing terraform-config-inspect
suzuki-shunsuke Mar 16, 2024
affbf63
fix: remove an input terraform_config_inspect_path
suzuki-shunsuke Mar 16, 2024
8b1f1ca
Use jest
exoego Mar 16, 2024
914aaed
Merge remote-tracking branch 'origin/update-module-use-site' into upd…
exoego Mar 16, 2024
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions check-terraform-skip/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions export-aws-secrets-manager/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions get-global-config/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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:
Expand Down
20 changes: 12 additions & 8 deletions get-global-config/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 4 additions & 0 deletions get-global-config/src/run.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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;
Expand Down Expand Up @@ -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,
Expand Down
3 changes: 3 additions & 0 deletions get-target-config/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

13 changes: 13 additions & 0 deletions lib/dist/index.d.ts

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

3 changes: 3 additions & 0 deletions lib/dist/index.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

5 changes: 5 additions & 0 deletions lib/src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -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()),
Expand Down
1 change: 1 addition & 0 deletions list-module-callers/.nvmrc
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
20.11.0
15 changes: 15 additions & 0 deletions list-module-callers/action.yaml
Original file line number Diff line number Diff line change
@@ -0,0 +1,15 @@
name: List module callers
description: List module's callers
inputs:
config_files:
description: "config file paths"
required: true
module_files:
description: "module file paths"
required: true
outputs:
file:
description: A json to list module callers
runs:
using: "node20"
main: "dist/index.js"
145 changes: 145 additions & 0 deletions list-module-callers/dist/LICENSE

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

Loading