-
Notifications
You must be signed in to change notification settings - Fork 27
[TFDN-182] TFE ToolVersion API Doc Updates for ARM Support #547
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
base: main
Are you sure you want to change the base?
Conversation
Vercel Previews Deployed
|
Broken Link CheckerNo broken links found! 🎉 |
46114ec
to
9384b3b
Compare
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.
I made some suggestions and linked to the style guide for my rationale. But just to sum up, I don't believe we have a concept of a "pre-deprecation" cycle, so until something is actually deprecated, I think the most we can say is that something is under review. "Deprecated soon" is too much of a hedge.
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/sentinel-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/sentinel-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/sentinel-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/sentinel-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
content/terraform-enterprise/v202506-1/docs/enterprise/api-docs/admin/terraform-versions.mdx
Outdated
Show resolved
Hide resolved
…s/admin/sentinel-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/sentinel-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/sentinel-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/terraform-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/terraform-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/terraform-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
…s/admin/terraform-versions.mdx Co-authored-by: trujillo-adam <[email protected]>
6cef466
to
d69f60c
Compare
d69f60c
to
d587190
Compare
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!
This PR updates the tool version API docs to reflect the addition of the
archs
field and the recommendation ofarch[n].url
andarch[n].sha
usage over the soon to be deprecated fields,url
andsha
.Jira
RFC