Skip to content
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

Removes the provider setting and improves the Readme #15

Merged
merged 4 commits into from
Jan 26, 2022

Conversation

china-1947
Copy link

Modified author contact information

README.md Outdated
| Name | Version |
|------|---------|
| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 0.12.0 |
| <a name="requirement_alicloud"></a> [alicloud](#requirement\_alicloud) | >= 1.56.0
Copy link
Contributor

Choose a reason for hiding this comment

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

从上面157行看,这个module的最小版本是 1.77.0

@@ -154,11 +154,78 @@ If you want to store kube config and other certificates after the cluster create

Terraform version
-----------------
Terraform version 0.12.0 or newer and Provider version 1.77.0 or newer are required for this example to work.
Copy link
Contributor

Choose a reason for hiding this comment

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

既然已经移除掉了provider,那么readme上方的Usage和Inputs中相关的字段内容也要去掉

@xiaozhu36 xiaozhu36 changed the title Modified author contact information Removes the provider setting and improves the Readme Nov 15, 2021
@xiaozhu36 xiaozhu36 merged commit 24f3458 into alibabacloud-automation:master Jan 26, 2022
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.

2 participants