-
Notifications
You must be signed in to change notification settings - Fork 9.8k
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
include help entry for -replace #36780
base: main
Are you sure you want to change the base?
Conversation
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.
This looks great. We need to add this flag to the docs as well:
website/docs/cli/commands/apply.mdx
under Apply Options
Ill make those changes now - thanks! |
Nice catch, and thanks for your contribution! I've added the 'no-changelog' label as we don't normally need changelogs for website or documentation changes. It would be great if you could remove the changelog file. Sorry for the confusion |
no problem, done :) thanks @dbanck |
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.
LGTM, just one minor whitespace fix
-replace=resource Terraform will plan to replace this resource instance | ||
instead of doing an update or no-op action. | ||
|
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.
-replace=resource Terraform will plan to replace this resource instance | |
instead of doing an update or no-op action. | |
-replace=resource Terraform will plan to replace this resource instance | |
instead of doing an update or no-op action. | |
I noticed a slight offset when printing the help text. Adding a space should fix it
- `-replace=resource` - Terraform will plan to replace this resource instance | ||
instead of doing an update or no-op action. | ||
|
||
|
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.
Removing the extra blank line
Fixes #36779
Target Release
1.12.x
CHANGELOG entry