Fix for issue #52 and potentially #62

sl1pm4t / k2tf

Kubernetes YAML to Terraform HCL converter

Mozilla Public License 2.0

1.17k stars 108 forks source link

Fix for issue #52 and potentially #62 #65

Closed tomaspinho closed 4 years ago

sl1pm4t commented 4 years ago

Thanks @tomaspinho ! Any chance you could add a unit test in hcl_writer_test.go to validate the fix?

tomaspinho commented 4 years ago

Hey @sl1pm4t ! Added the test cases, however the tests keep failing because of non-determinism in the resulting key order.

sl1pm4t commented 4 years ago

Thanks @tomaspinho - yes, the non-deterministic key order is a known problem. It got worse recently with a golang version bump (I think). Thanks for the fix and the test!