Skip to content

refactor 'provisioners' content #699

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

Open
wants to merge 1 commit into
base: docs/reference-rewrites-phrase-1
Choose a base branch
from

Conversation

trujillo-adam
Copy link
Contributor

This PR merges the non-reference content for using provisioners into a single usage page. It also combines guidance on using alternatives to provisioners. The resulting topic is about performing post-apply operations. Note that I kept the file name provisioners.mdx for a few reasons:

  1. To make redirecting from the removed subpages easier.
  2. So that the keyword remains in the URL so that people who are specifically looking for provisioners can more easily find the page through search.
  3. No consensus on an alternative.

I added a redirect and tried to fix all links, but we will need to take another pass after all of the content has been merged to the rollup branch.

For this review, just look for copy errors and obvious content problems. During the draft phase, there were a lot of technical questions. In some cases, I was able to address them all. In others, I reverted to original text on the assumption that if it was already there it had been verified in the past.

@trujillo-adam trujillo-adam marked this pull request as ready for review August 7, 2025 01:10
@trujillo-adam trujillo-adam requested review from a team as code owners August 7, 2025 01:10
Copy link

github-actions bot commented Aug 7, 2025

Vercel Previews Deployed

Name Status Preview Updated (UTC)
Dev Portal ✅ Ready (Inspect) Visit Preview Thu Aug 7 01:33:45 UTC 2025
Unified Docs API ✅ Ready (Inspect) Visit Preview Thu Aug 7 01:26:12 UTC 2025

Copy link

github-actions bot commented Aug 7, 2025

Broken Link Checker

Summary

Status Count
🔍 Total 285
✅ Successful 264
⏳ Timeouts 0
🔀 Redirected 0
👻 Excluded 0
❓ Unknown 0
🚫 Errors 21

Errors per input

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

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

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

Errors in content/terraform/v1.12.x/docs/language/state/sensitive-data.mdx

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant