Make upsert_object more readeable by splitting it

opsmill / infrahub-demo-dc-fabric-develop

2 stars 4 forks source link

Make upsert_object more readeable by splitting it #22

Closed BeArchiTek closed 6 months ago

BeArchiTek commented 6 months ago

Split the function upsert_object into two functions :

create_and_save()
create_and_add_to_batch()

Cleanup some of the utils functions already merged into the SDK

wvandeun commented 6 months ago

There seems to be 16 commits in this PR, but only 3 of them seem to contribute to the end-result of the PR. Can we clean this up, because it is confusing and is going to be more so if we ever need to come back to this? Maybe need to rebase a few things?

BeArchiTek commented 6 months ago

Replace by https://github.com/opsmill/infrahub-demo-dc-fabric-develop/pull/30 for better readability