New Resource: azurerm_data_factory_dataset_sql_server_table
#3236
Conversation
| return &annotations | ||
| } | ||
|
|
||
| func flattenDataFactoryAnnotations(input *[]interface{}) []string { |
There was a problem hiding this comment.
I think you could use utils.FlattenStringArray instead of this?
There was a problem hiding this comment.
Weirdly enough, I think it'd be a call to utils.ExpandStringArray
There was a problem hiding this comment.
On second thought. I think it's needed because it's an *[]interface{} and we'd need to do some finagling to get it to a []string
There was a problem hiding this comment.
Ahh yea sorry, its so close to the signature, and as much as i'd want to ask if terraform would accept the interface array i know we should check they are all strings.
azurerm/resource_arm_data_factory_dataset_sql_server_table_test.go
Outdated
Show resolved
Hide resolved
azurerm/resource_arm_data_factory_dataset_sql_server_table_test.go
Outdated
Show resolved
Hide resolved
azurerm/resource_arm_data_factory_dataset_sql_server_table_test.go
Outdated
Show resolved
Hide resolved
azurerm/resource_arm_data_factory_dataset_sql_server_table_test.go
Outdated
Show resolved
Hide resolved
|
I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues. If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 hashibot-feedback@hashicorp.com. Thanks! |
ghost
locked and limited conversation to collaborators
This PR adds
azurerm_data_factory_dataset_sql_server_tableto the Data Factory collection and starts to consolidate some of the methods used between the resources into a single source underdatafactory.go