From fed6538dece2b0306d52f6c12facb5c4e5485725 Mon Sep 17 00:00:00 2001 From: Rees Pozzi Date: Fri, 5 May 2023 21:36:05 +0100 Subject: [PATCH] core: Correct typo in "Moved resource instances excluded by targeting" error message --- internal/terraform/context_plan.go | 2 +- internal/terraform/context_plan2_test.go | 6 +++--- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/internal/terraform/context_plan.go b/internal/terraform/context_plan.go index 3ade28c1ce62..64f214d7c4b8 100644 --- a/internal/terraform/context_plan.go +++ b/internal/terraform/context_plan.go @@ -500,7 +500,7 @@ func (c *Context) prePlanVerifyTargetedMoves(moveResults refactoring.MoveResults tfdiags.Error, "Moved resource instances excluded by targeting", fmt.Sprintf( - "Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource instances.\n\nTo create a valid plan, either remove your -target=... options altogether or add the following additional target options:%s\n\nNote that adding these options may include further additional resource instances in your plan, in order to respect object dependencies.", + "Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options do not fully cover all of those resource instances.\n\nTo create a valid plan, either remove your -target=... options altogether or add the following additional target options:%s\n\nNote that adding these options may include further additional resource instances in your plan, in order to respect object dependencies.", listBuf.String(), ), )) diff --git a/internal/terraform/context_plan2_test.go b/internal/terraform/context_plan2_test.go index 3d62058bc07b..303cf3b9fc26 100644 --- a/internal/terraform/context_plan2_test.go +++ b/internal/terraform/context_plan2_test.go @@ -1574,7 +1574,7 @@ The -target option is not for routine use, and is provided only for exceptional tfdiags.Sourceless( tfdiags.Error, "Moved resource instances excluded by targeting", - `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource instances. + `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options do not fully cover all of those resource instances. To create a valid plan, either remove your -target=... options altogether or add the following additional target options: -target="test_object.a" @@ -1614,7 +1614,7 @@ The -target option is not for routine use, and is provided only for exceptional tfdiags.Sourceless( tfdiags.Error, "Moved resource instances excluded by targeting", - `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource instances. + `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options do not fully cover all of those resource instances. To create a valid plan, either remove your -target=... options altogether or add the following additional target options: -target="test_object.b" @@ -1654,7 +1654,7 @@ The -target option is not for routine use, and is provided only for exceptional tfdiags.Sourceless( tfdiags.Error, "Moved resource instances excluded by targeting", - `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options to not fully cover all of those resource instances. + `Resource instances in your current state have moved to new addresses in the latest configuration. Terraform must include those resource instances while planning in order to ensure a correct result, but your -target=... options do not fully cover all of those resource instances. To create a valid plan, either remove your -target=... options altogether or add the following additional target options: -target="test_object.a"