Remove terraform-plugin-sdk (v1) support

This dependency was marked end-of-life many years ago and it causes issues with attempting to support newer versions of terraform-plugin-sdk/v2 due to shared dependencies with breaking changes. This change removes the v1 dependency and empties out any analyzers with logic solely for that version. The analyzers will remain in place to reserve the analyzer name, but they will now never return reports.

README.md

@@ -118,7 +118,7 @@ Standard lint checks are enabled by default in the `tfproviderlint` tool. Opt-in
 | [R005](passes/R005) | check for `ResourceData.HasChange()` calls that can be combined into one `HasChanges()` call | AST |
 | [R006](passes/R006) | check for `RetryFunc` that omit retryable errors | AST |
 | [R007](passes/R007) | check for deprecated `(schema.ResourceData).Partial` usage | AST |
-| [R008](passes/R008) | check for deprecated `(schema.ResourceData).SetPartial` usage | AST |
+| [R008](passes/R008) | **REMOVED in v0.30.0** check for deprecated `(schema.ResourceData).SetPartial` usage | AST |
 | [R009](passes/R009) | check for Go panic usage | AST |
 | [R010](passes/R010) | check for `(schema.ResourceData).GetChange` assignment which should use `(schema.ResourceData).Get` | AST |
 | [R011](passes/R011) | check for `Resource` that configure `MigrateState` | AST |
@@ -168,7 +168,7 @@ Standard lint checks are enabled by default in the `tfproviderlint` tool. Opt-in
 | [S031](passes/S031) | check for `Schema` of `Computed` only with `MaxItems` configured | AST |
 | [S032](passes/S032) | check for `Schema` of `Computed` only with `MinItems` configured | AST |
 | [S033](passes/S033) | check for `Schema` of `Computed` only with `StateFunc` configured | AST |
-| [S034](passes/S034) | check for `Schema` that configure `PromoteSingle` | AST |
+| [S034](passes/S034) | **REMOVED in v0.30.0** check for `Schema` that configure `PromoteSingle` | AST |
 | [S035](passes/S035) | check for `Schema` with invalid `AtLeastOneOf` attribute references | AST |
 | [S036](passes/S036) | check for `Schema` with invalid `ConflictsWith` attribute references | AST |
 | [S037](passes/S037) | check for `Schema` with invalid `ExactlyOneOf` attribute references | AST |
@@ -178,13 +178,13 @@ Standard lint checks are enabled by default in the `tfproviderlint` tool. Opt-in
 | Check | Description | Type |
 |---|---|---|
 | [V001](passes/V001) | check for custom `SchemaValidateFunc` that implement `validation.StringMatch()` or `validation.StringDoesNotMatch()` | AST |
-| [V002](passes/V002) | check for deprecated `CIDRNetwork` validation function usage | AST |
-| [V003](passes/V003) | check for deprecated `IPRange` validation function usage | AST |
-| [V004](passes/V004) | check for deprecated `SingleIP` validation function usage | AST |
-| [V005](passes/V005) | check for deprecated `ValidateJsonString` validation function usage | AST |
-| [V006](passes/V006) | check for deprecated `ValidateListUniqueStrings` validation function usage | AST |
-| [V007](passes/V007) | check for deprecated `ValidateRegexp` validation function usage | AST |
-| [V008](passes/V008) | check for deprecated `ValidateRFC3339TimeString` validation function usage | AST |
+| [V002](passes/V002) | **REMOVED in v0.30.0** check for deprecated `CIDRNetwork` validation function usage | AST |
+| [V003](passes/V003) | **REMOVED in v0.30.0** check for deprecated `IPRange` validation function usage | AST |
+| [V004](passes/V004) | **REMOVED in v0.30.0** check for deprecated `SingleIP` validation function usage | AST |
+| [V005](passes/V005) | **REMOVED in v0.30.0** check for deprecated `ValidateJsonString` validation function usage | AST |
+| [V006](passes/V006) | **REMOVED in v0.30.0** check for deprecated `ValidateListUniqueStrings` validation function usage | AST |
+| [V007](passes/V007) | **REMOVED in v0.30.0** check for deprecated `ValidateRegexp` validation function usage | AST |
+| [V008](passes/V008) | **REMOVED in v0.30.0** check for deprecated `ValidateRFC3339TimeString` validation function usage | AST |
 | [V009](passes/V009) | check for `validation.StringMatch()` call with empty message argument | AST |
 | [V010](passes/V010) | check for `validation.StringDoesNotMatch()` call with empty message argument | AST |
 | [V011](passes/V011) | check for custom `SchemaValidateFunc` that implement `validation.StringLenBetween()` | AST |

go.mod

@@ -4,31 +4,24 @@ go 1.21
 
 require (
 	github.com/bflad/gopaniccheck v0.1.0
-	github.com/hashicorp/terraform-plugin-sdk v1.17.2
 	github.com/hashicorp/terraform-plugin-sdk/v2 v2.5.0
 	golang.org/x/tools v0.21.0
 )
 
 require (
 	cloud.google.com/go v0.65.0 // indirect
 	cloud.google.com/go/storage v1.10.0 // indirect
-	github.com/Masterminds/goutils v1.1.0 // indirect
-	github.com/Masterminds/semver v1.5.0 // indirect
-	github.com/Masterminds/sprig v2.22.0+incompatible // indirect
 	github.com/agext/levenshtein v1.2.2 // indirect
 	github.com/apparentlymart/go-cidr v1.1.0 // indirect
 	github.com/apparentlymart/go-textseg/v12 v12.0.0 // indirect
 	github.com/apparentlymart/go-textseg/v13 v13.0.0 // indirect
-	github.com/armon/go-radix v1.0.0 // indirect
 	github.com/aws/aws-sdk-go v1.37.0 // indirect
 	github.com/bgentry/go-netrc v0.0.0-20140422174119-9fd32a8b3d3d // indirect
-	github.com/bgentry/speakeasy v0.1.0 // indirect
 	github.com/davecgh/go-spew v1.1.1 // indirect
 	github.com/fatih/color v1.7.0 // indirect
 	github.com/golang/groupcache v0.0.0-20200121045136-8c9f03a8e57e // indirect
 	github.com/golang/protobuf v1.4.2 // indirect
 	github.com/google/go-cmp v0.6.0 // indirect
-	github.com/google/uuid v1.1.2 // indirect
 	github.com/googleapis/gax-go/v2 v2.0.5 // indirect
 	github.com/hashicorp/errwrap v1.0.0 // indirect
 	github.com/hashicorp/go-checkpoint v0.5.0 // indirect
@@ -41,40 +34,27 @@ require (
 	github.com/hashicorp/go-safetemp v1.0.0 // indirect
 	github.com/hashicorp/go-uuid v1.0.1 // indirect
 	github.com/hashicorp/go-version v1.3.0 // indirect
-	github.com/hashicorp/hcl v0.0.0-20170504190234-a4b07c25de5f // indirect
 	github.com/hashicorp/hcl/v2 v2.8.2 // indirect
 	github.com/hashicorp/logutils v1.0.0 // indirect
-	github.com/hashicorp/terraform-config-inspect v0.0.0-20191212124732-c6ae6269b9d7 // indirect
 	github.com/hashicorp/terraform-exec v0.13.3 // indirect
 	github.com/hashicorp/terraform-json v0.10.0 // indirect
 	github.com/hashicorp/terraform-plugin-go v0.2.1 // indirect
-	github.com/hashicorp/terraform-plugin-test/v2 v2.2.1 // indirect
-	github.com/hashicorp/terraform-svchost v0.0.0-20200729002733-f050f53b9734 // indirect
 	github.com/hashicorp/yamux v0.0.0-20181012175058-2f1d1f20f75d // indirect
-	github.com/huandu/xstrings v1.3.2 // indirect
-	github.com/imdario/mergo v0.3.12 // indirect
 	github.com/jmespath/go-jmespath v0.4.0 // indirect
 	github.com/jstemmer/go-junit-report v0.9.1 // indirect
 	github.com/klauspost/compress v1.11.2 // indirect
 	github.com/mattn/go-colorable v0.1.4 // indirect
 	github.com/mattn/go-isatty v0.0.10 // indirect
-	github.com/mitchellh/cli v1.1.2 // indirect
-	github.com/mitchellh/colorstring v0.0.0-20190213212951-d06e56a500db // indirect
 	github.com/mitchellh/copystructure v1.0.0 // indirect
 	github.com/mitchellh/go-homedir v1.1.0 // indirect
 	github.com/mitchellh/go-testing-interface v1.0.4 // indirect
 	github.com/mitchellh/go-wordwrap v1.0.0 // indirect
 	github.com/mitchellh/mapstructure v1.1.2 // indirect
 	github.com/mitchellh/reflectwalk v1.0.1 // indirect
 	github.com/oklog/run v1.0.0 // indirect
-	github.com/posener/complete v1.2.1 // indirect
-	github.com/spf13/afero v1.2.2 // indirect
 	github.com/ulikunitz/xz v0.5.8 // indirect
 	github.com/vmihailenco/msgpack v4.0.4+incompatible // indirect
-	github.com/vmihailenco/msgpack/v4 v4.3.12 // indirect
-	github.com/vmihailenco/tagparser v0.1.1 // indirect
 	github.com/zclconf/go-cty v1.8.2 // indirect
-	github.com/zclconf/go-cty-yaml v1.0.2 // indirect
 	go.opencensus.io v0.22.4 // indirect
 	golang.org/x/crypto v0.23.0 // indirect
 	golang.org/x/lint v0.0.0-20200302205851-738671d3881b // indirect