Skip to content

Conversation

@niejn
Copy link

@niejn niejn commented Jan 20, 2022

I am using python-terraform in my projects, and I found out that python-terraform lib is not supporting latest terraform, such ass v1.1.0. So I want to contribute to add terraform v1.1.0 support

@jerger
Copy link

jerger commented Jan 20, 2022

We have already done the work on our fork here:
https://gitlab.com/domaindrivenarchitecture/python-terraform

As we moved to gitlab (doe to nicer ci-support) a PullRequest will be slightly more work.
But we will help to reunite the changes :-)

@Th0masL
Copy link

Th0masL commented Jul 9, 2022

Hey guys! Is there any extra help needed to be able to merge those changes? The support for the version 1.x would be pretty awsome !

@jerger
Copy link

jerger commented Jul 12, 2022

At https://gitlab.com/domaindrivenarchitecture/python-terraform tf > 1 is working. But we've done more changes than just the lift to 1.0. So extra help would be very apreciated :-)

We can do a mob-pr session to setup a PR - in case of interest you can contact me at [email protected]

@Th0masL
Copy link

Th0masL commented Jul 14, 2022

Sadly its not very convenient to deal with changes/PRs between github and gitlab.

My main question is: does this PR contains everything you guys have added to your gitlab repo ?

If the answer is no, I'm happy to make a new PR that includes more stuff, just let me know what you think is the best approach @jerger, thanks !

@jerger
Copy link

jerger commented Jul 14, 2022

No, we've not contributed to this PR - our solution would produce conflicts.

@Th0masL
Copy link

Th0masL commented Jul 15, 2022

Thanks for your answer @jerger! I've had a look at your changes and I've created a new branch based on the content of your gitlab repo, and I also tried to resolve most of the minor differences to try to make the reunion of the two repos as easy as posisble.

Here is the branch: https://github.com/Th0masL/python-terraform/tree/1.0.4-dev-dda

Is there more things we need to pair on before this would eventually get merged ?

@jerger
Copy link

jerger commented Jul 16, 2022

No let's create a PR as work in progress & discuss about our changes while seeing the diffs.

@Th0masL
Copy link

Th0masL commented Jul 17, 2022

#123

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants