-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
docs: Add doc notes regards tf v1.3 support #1010
Conversation
@@ -105,6 +105,9 @@ If you are interested in contributing to EKS Blueprints, see the [Contribution g | |||
|
|||
--- | |||
|
|||
> **_NOTE:_** Use terraform versions after version 1.0.0 and before version 1.3.0. |
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.
terraform-docs will wipe this out (after CI checks complain)
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.
it's before the <!-- BEGINNING OF PRE-COMMIT-TERRAFORM DOCS HOOK -->
, you sure?
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.
oh .... I'm not that smart, please ignore me 😬
Likely not needed after #1011 |
Using terraform 1.3 is still a problem we need to document for user, they can't use 1.3 cc @bryantbiggs |
don't know why the md lint is failing is just one line change |
its this 7abef5b we can ignore for now and once my PR lands it will be fixed |
ah like something got moved, and the current md is broken in main |
Co-authored-by: Carlos Santana <[email protected]>
What does this PR do?
🛑 Please open an issue first to discuss any significant work and flesh out details/direction - we would hate for your time to be wasted.
Consult the CONTRIBUTING guide for submitting pull-requests.
Motivation
More
pre-commit run -a
with this PRNote: Not all the PRs require a new example and/or doc page. In general:
docs/add-ons/*
is required for new a new addonFor Moderators
Additional Notes