cdktf / cdktf-tf-module-stack

A drop-in replacement for cdktf.TerraformStack that lets you define Terraform modules as constructs
Mozilla Public License 2.0
20 stars 2 forks source link

Undefined type in go package: AppOptions #246

Closed FooBartn closed 1 year ago

FooBartn commented 1 year ago

The go package for this will not build due to an undefined type.

../../../../../../go/pkg/mod/github.com/cdktf/cdktf-tf-module-stack-go/tfmodulestack@v1.1.21/tfmodulestack_TFModuleApp.go:96:36: undefined: cdktf.AppOptions
../../../../../../go/pkg/mod/github.com/cdktf/cdktf-tf-module-stack-go/tfmodulestack@v1.1.21/tfmodulestack_TFModuleApp.go:113:60: undefined: cdktf.AppOptions
../../../../../../go/pkg/mod/github.com/cdktf/cdktf-tf-module-stack-go/tfmodulestack@v1.1.21/tfmodulestack_TFModuleApp__runtime_type_checks.go:55:54: undefined: cdktf.AppOptions

Looking at the other language versions and the cdtkf, this should be cdktf.AppConfig, I believe.

FooBartn commented 1 year ago

Closing, as the real issue is that the Readme should be updated to instruct the use of

go get github.com/cdktf/cdktf-tf-module-stack-go/tfmodulestack/v2 instead of go get github.com/cdktf/cdktf-tf-module-stack-go/tfmodulestack