-
Notifications
You must be signed in to change notification settings - Fork 4
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
Enable Publishing to Terraform Registry #5
Conversation
5a3d239
to
68f6a7d
Compare
# Visit https://golangci-lint.run/ for usage documentation | ||
# and information on other useful linters |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think this a stray file and the golangci lint changed a lot after this. We can delete this one.
flags: | ||
- -trimpath | ||
ldflags: | ||
- '-s -w -X main.version={{.Version}} -X main.commit={{.Commit}}' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see the main.commit
in the main package, but I do see the version.
* add outputs to support airgap I want to be able to build route53 records off of stateful ids and the "public" ip generated by a supplied vpc. Doing this with the current outputs results in issues when using random_id + count on the module's outputs. With this method we should be ok. * minor fix + test
Pretty much exactly follows the directions here: https://developer.hashicorp.com/terraform/registry/providers/publishing
Only differences are golangci remediation.