Closed austinvalle closed 6 months ago
terraform-plugin-docs has been updated on main 👍
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. If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.
Ref: https://github.com/hashicorp/terraform-provider-time/issues/217 Terraform core refs: https://github.com/hashicorp/terraform/issues/29694, https://github.com/hashicorp/terraform/issues/31751, https://github.com/hashicorp/terraform/issues/25802, https://github.com/hashicorp/terraform/issues/33538
This PR introduces a new provider-defined function for parsing RFC3339 timestamps:
Notes
(time).Zone()
zone_name
is not directly related tozone_offset
, and would always be"UTC"
or""
zone_offset
is valid and a part of the RFC3339 string. Thetime.Time
package returns this offset in seconds east of UTC, which may be confusing to users (but potentially necessary in this format, since time zone offsets aren't necessarily in whole hours 😖 ).(time).ParseInLocation
function_
to follow the convention of the repository. I can also refactor all the files in this provider to match the other accepted pattern of:*_function.go
*_resource.go