Baraa Basata

Results 10 comments of Baraa Basata

> Would it be possible to move the `#cloud-init` above the `--MIMEBOUNDARY`, or add an additional one there? Hello, and thank you for your patience with the delay in reviewing...

The failing ci-go checks are unrelated to this change. I will re-run the checks for this PR after we fix the other tests on `main`.

Hi @sestegra – I'm a maintainer and I'd like to assist with merging this change. As I commented in https://github.com/hashicorp/terraform-provider-tls/pull/309#issuecomment-3070947692, I would like to merge this independently of other changes....

> @bbasata Rebase done Workflows approved ⏩

> @bbasata I fixed the compilation issue, but not sure about if it's the expected behavior Workflows approved ⏩ I will take a closer look at the change ✅

> @bbasata I fixed the compilation issue, but not sure about if it's the expected behavior @sestegra: to set the default value, you can add this to the attributes: ```...

> @bbasata I added the default values. Excellent! I will try to review & merge sometime this week. In the meantime, would you please [add a changelog entry](https://github.com/hashicorp/terraform-provider-tls/blob/main/.github/CONTRIBUTING.md#changelog)?

Related: `list.ListResource` as a stand-alone interface in https://github.com/hashicorp/terraform-plugin-framework/pull/1150. @gdavison: I invite you to review the types and the discussion while this is still fresh & pliable 😃 > A developer...

> may I get some thoughts from your team about this proposal? Thank you! Hi @ChihweiLHBird, Thank you for your patience. The team recently selected this pull request for triage,...

> It's nice to have Length function for collection types (set, list, map, etc.), so we won't have to do `len(list.Elements())` which is wasting time to allocate and release memory...