search

DopplerHQ / terraform-provider-doppler

Apache License 2.0
23 stars 9 forks source link

Ignore order of project_member environments #79

Closed rgharris closed 7 months ago

rgharris commented 7 months ago

Release note: Utilize a TypeSet. No change in usage.

Force push squashes the extra "remove" commit which was left initially for review purposes only.

Closes ENG-7578 Closes #73

rgharris commented 7 months ago

Nice! I thought this was going to be a lot tricker.

Same!

I did some light local testing, including the scenario where we have an existing project member resource in state and we upgrade to using this new provider version. It seems that Terraform serializes both lists and sets as JSON arrays under the hood, so it just works ✨

I initially wrote a migration but it turned out to be unnecessary due to the serialization as you note so I removed it. I tried to leave it in here for your review but apparently Github removed it after some amount of time so you can't see the force push comparison 🤷.