Skip to content

added removed block reference #508

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

Merged
merged 13 commits into from
Jul 2, 2025

Conversation

trujillo-adam
Copy link
Contributor

@trujillo-adam trujillo-adam commented Jun 19, 2025

This PR adds the reference page for the removed block. It also converts the provisioner connection settings table into a partial.

@trujillo-adam trujillo-adam marked this pull request as ready for review June 19, 2025 22:18
@trujillo-adam trujillo-adam requested review from a team as code owners June 19, 2025 22:18
@trujillo-adam trujillo-adam added documentation Improvements or additions to documentation Terraform labels Jun 19, 2025
@trujillo-adam trujillo-adam reopened this Jun 24, 2025
@trujillo-adam trujillo-adam changed the base branch from docs/reference-rewrites-phrase-1 to main June 25, 2025 17:04
@trujillo-adam trujillo-adam changed the base branch from main to docs/reference-rewrites-phrase-1 June 25, 2025 17:09
@trujillo-adam trujillo-adam added link-checker-report documentation Improvements or additions to documentation Terraform and removed documentation Improvements or additions to documentation Terraform link-checker-report labels Jun 25, 2025
@trujillo-adam trujillo-adam changed the base branch from docs/reference-rewrites-phrase-1 to main June 30, 2025 20:31
@trujillo-adam trujillo-adam changed the base branch from main to docs/reference-rewrites-phrase-1 June 30, 2025 20:36
@trujillo-adam trujillo-adam requested review from a team as code owners July 1, 2025 23:14
@trujillo-adam trujillo-adam requested review from RubenSandwich and removed request for a team July 1, 2025 23:14
Copy link

hashicorp-cla-app bot commented Jul 1, 2025

CLA assistant check
All committers have signed the CLA.

@RubenSandwich RubenSandwich force-pushed the docs/reference-rewrites-phrase-1 branch from 0124df5 to 2a6609e Compare July 2, 2025 14:27
Copy link

github-actions bot commented Jul 2, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Wed Jul 2 21:12:40 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Wed Jul 2 21:07:47 UTC 2025

Copy link

github-actions bot commented Jul 2, 2025

Broken Link Checker

Summary

Status Count
🔍 Total 272
✅ Successful 259
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 13

Errors per input

Errors in content/terraform/v1.12.x/docs/language/v1-compatibility-promises.mdx

Errors in content/terraform/v1.12.x/docs/language/block/resource.mdx

Errors in content/terraform/v1.12.x/docs/language/style.mdx

Errors in content/terraform/v1.12.x/docs/language/block/removed.mdx

Copy link
Contributor

@robin-norwood robin-norwood left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!


# `removed` block reference

The `removed` block specifies a resource to remove from state. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on removing resources from both state and configuration.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The `removed` block specifies a resource to remove from state. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on removing resources from both state and configuration.
The `removed` block specifies a resource to remove from state. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on destroying resources by removing them from both state and configuration.

This is talking about the use case of "I want to destroy resources that are managed by Terraform, right?

Copy link
Contributor Author

@trujillo-adam trujillo-adam Jul 2, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is talking about the use case of "I want to destroy resources that are managed by Terraform, right?

This is the use case where you no longer want to manage the resource with Terraform. Here is the current draft.


# `removed` block reference

The `removed` block specifies a resource to remove from state. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on removing resources from both state and configuration.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
The `removed` block specifies a resource to remove from state. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on removing resources from both state and configuration.
The `removed` block specifies a resource to remove from state without making changes to the underlying infrastructure. Refer to [Remove a resource](/terraform/language/resources/remove)<!--URL TBD --> for instructions on removing resources from both state and configuration.

@trujillo-adam trujillo-adam merged commit 1e6732d into docs/reference-rewrites-phrase-1 Jul 2, 2025
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation Terraform
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants