Skip to content

Replace custom conditions article with Validate your configuration #36793

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

Conversation

rkoron007
Copy link
Contributor

@rkoron007 rkoron007 commented Mar 27, 2025

This is part of our IA work -> making this a usage article around the J2BD of "validating your configuration".

@rkoron007 rkoron007 changed the base branch from main to docs/march-sprint-lang-rewrites March 27, 2025 17:18
@digital-content-events
Copy link

📄 Content Checks

Updated: Thu, 27 Mar 2025 17:18:24 GMT

Found 7 error(s)

docs/language/mitigate-risk/validate-config.mdx

Position Description Rule
98:361-98:401 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
115:127-115:165 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
139:96-139:138 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
186:32-186:71 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
376:3-376:42 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
377:3-377:45 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format
378:3-378:45 Unexpected folder-relative link found: TODO. Ensure this link is an absolute Developer path. ensure-valid-link-format

@rkoron007 rkoron007 force-pushed the RK/validate-config-rewrite branch from dd24e83 to 69c0367 Compare March 27, 2025 17:46
@rkoron007 rkoron007 added the no-changelog-needed Add this to your PR if the change does not require a changelog entry label Mar 27, 2025
@rkoron007 rkoron007 changed the title Rewrite validate config Replace custom conditions article with Validate your configuration Mar 27, 2025
@rkoron007 rkoron007 self-assigned this Mar 27, 2025
@@ -13,7 +13,7 @@ This topic provides reference information about the `terraform` block. The `terr
The `terraform` block supports the following arguments:

- [`terraform`](#terraform)
- [`required_version`](#required_version): &nbsp string
- [`required_version`](#required_version): &nbsp string
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sorry this is just my linter cleaning up whitespace -> no changes here

@@ -78,3 +78,134 @@ a string.
```hcl
var.example ? tostring(12) : "hello"
```

## Examples
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Moving this all from the old "Custom Conditions" page I'm replacing, and it felt like a natural fit here!

@rkoron007 rkoron007 marked this pull request as ready for review March 27, 2025 22:23
@rkoron007 rkoron007 requested review from a team as code owners March 27, 2025 22:23
Copy link
Contributor

@trujillo-adam trujillo-adam left a comment

Choose a reason for hiding this comment

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

Looks great! I added some suggestions mostly related to the style guide. I don't think anything should hold this up, though, so take what you agree with.

@rkoron007 rkoron007 deleted the branch docs/reference-rewrites-phrase-1 March 31, 2025 22:55
@rkoron007 rkoron007 closed this Mar 31, 2025
@rkoron007 rkoron007 reopened this Mar 31, 2025
@rkoron007 rkoron007 changed the base branch from docs/march-sprint-lang-rewrites to docs/reference-rewrites-phrase-1 March 31, 2025 23:08
@rkoron007 rkoron007 merged commit 9b271ad into docs/reference-rewrites-phrase-1 Apr 1, 2025
8 checks passed
@rkoron007 rkoron007 deleted the RK/validate-config-rewrite branch April 1, 2025 23:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no-changelog-needed Add this to your PR if the change does not require a changelog entry
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants