-
Notifications
You must be signed in to change notification settings - Fork 20
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
docs: v1 to v2 upgrade guide #259
Conversation
3ece335
to
e7a4b02
Compare
601b056
to
cd46d6d
Compare
cd46d6d
to
4163803
Compare
4163803
to
1efcd77
Compare
@@ -8,7 +8,11 @@ description: |- | |||
|
|||
# Coder Provider | |||
|
|||
Terraform provider for managing Coder [templates](https://coder.com/docs/templates), which are the underlying infrastructure for Coder [workspaces](https://coder.com/docs/workspaces). | |||
Terraform provider for managing Coder [templates](https://coder.com/docs/admin/templates), which are the underlying infrastructure for Coder [workspaces](https://coder.com/docs/user-guides/workspace-management). |
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.
These links could be updated when we have a Glossary of terms.
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.
Copilot reviewed 5 out of 5 changed files in this pull request and generated no suggestions.
|
||
## Provider Arguments | ||
|
||
Version 2.0.0 removes the [`feature_use_managed_variables`](https://registry.terraform.io/providers/coder/coder/1.0.4/docs#feature_use_managed_variables-1) argument from the `provider` block. |
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.
Maybe we should explain that this has been deprecated for a long time and has no impact on Coder workspace anymore.
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.
All LGTM!
a7b09d2
to
87ad4f7
Compare
Adds the upgrade guide for v1 to v2.
I could not find an easier way to preview and test the guide, but it should look like the recording below.
courtesy: https://registry.terraform.io/tools/doc-preview
Screen.Recording.2024-11-21.at.6.33.00.PM.mov