-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
chore - use auto-generated parsers and validators for resource azurerm_machine_learning_workspace
#11739
chore - use auto-generated parsers and validators for resource azurerm_machine_learning_workspace
#11739
Conversation
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.
Thanks @gro1m - LGTM 👍
@katbyte Thanks you as well:) Should have covered linter issues now, i.e. I make fmt-ed the files in question. Unfortunately I cannot see a difference in the validator files of |
This has been released in version 2.60.0 of the provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading. As an example: provider "azurerm" {
version = "~> 2.60.0"
}
# ... other configuration ... |
…rm_machine_learning_workspace` (hashicorp#11739)
…rm_machine_learning_workspace` (hashicorp#11739)
…rm_machine_learning_workspace` (hashicorp#11739)
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |
This PR makes updates the current
azurerm_machine_learning_workspace
resource with using the capability to auto-generate the parsers and validators.This could have the maybe unwanted side-effect of dropping parsing of azure container registry and azure storage account validations. To be assessed by the reviewer and maybe small changes could improve this still a bit:)
Test should still pass: