Skip to content

Commit b52fcc8

Browse files
committed
Changed wording to install external dependency
Signed-off-by: Babur Ayanlar <[email protected]>
1 parent 480cad8 commit b52fcc8

File tree

2 files changed

+6
-6
lines changed

2 files changed

+6
-6
lines changed

internal/runner/common/unit_resolver_dependencies.go

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ import (
99
)
1010

1111
// flagExternalDependencies processes units that were marked as external by discovery,
12-
// prompting the user whether to apply them and setting appropriate flags.
12+
// prompting the user whether to install them and setting appropriate flags.
1313
// Discovery has already found, parsed, and marked external dependencies.
1414
// This function only handles the user-facing logic for deciding whether to run them.
1515
func (r *UnitResolver) flagExternalDependencies(ctx context.Context, l log.Logger, unitsMap UnitsMap) error {
@@ -43,7 +43,7 @@ func (r *UnitResolver) flagExternalDependencies(ctx context.Context, l log.Logge
4343
if dependentUnit != nil {
4444
var err error
4545

46-
shouldApply, err = r.confirmShouldApplyExternalDependency(ctx, dependentUnit, l, unit, unit.TerragruntOptions)
46+
shouldApply, err = r.confirmShouldInstallExternalDependency(ctx, dependentUnit, l, unit, unit.TerragruntOptions)
4747
if err != nil {
4848
return err
4949
}
@@ -62,11 +62,11 @@ func (r *UnitResolver) flagExternalDependencies(ctx context.Context, l log.Logge
6262
}
6363

6464
// Confirm with the user whether they want Terragrunt to assume the given dependency of the given unit is already
65-
// applied. If the user selects "yes", then Terragrunt will apply that unit as well.
65+
// applied. If the user selects "yes", then Terragrunt will install that unit as well.
6666
// Note that we skip the prompt for `run --all destroy` calls. Given the destructive and irreversible nature of destroy, we don't
6767
// want to provide any risk to the user of accidentally destroying an external dependency unless explicitly included
6868
// with the --queue-include-external or --queue-include-dir flags.
69-
func (r *UnitResolver) confirmShouldApplyExternalDependency(ctx context.Context, unit *Unit, l log.Logger, dependency *Unit, opts *options.TerragruntOptions) (bool, error) {
69+
func (r *UnitResolver) confirmShouldInstallExternalDependency(ctx context.Context, unit *Unit, l log.Logger, dependency *Unit, opts *options.TerragruntOptions) (bool, error) {
7070
if opts.IncludeExternalDependencies {
7171
l.Debugf("The --queue-include-external flag is set, so automatically including all external dependencies, and will run this command against unit %s, which is a dependency of unit %s.", dependency.Path, unit.Path)
7272
return true, nil
@@ -85,5 +85,5 @@ func (r *UnitResolver) confirmShouldApplyExternalDependency(ctx context.Context,
8585

8686
l.Infof("Unit %s has external dependency %s", unit.Path, dependency.Path)
8787

88-
return shell.PromptUserForYesNo(ctx, l, "Should Terragrunt apply the external dependency?", opts)
88+
return shell.PromptUserForYesNo(ctx, l, "Should Terragrunt install the external dependency?", opts)
8989
}

test/integration_destroy_test.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -342,7 +342,7 @@ func TestTerragruntSkipConfirmExternalDependencies(t *testing.T) {
342342
captured := <-capturedOutput
343343

344344
require.NoError(t, err)
345-
assert.NotContains(t, captured, "Should Terragrunt apply the external dependency?")
345+
assert.NotContains(t, captured, "Should Terragrunt install the external dependency?")
346346
assert.NotContains(t, captured, tmp)
347347
}
348348

0 commit comments

Comments
 (0)