Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Terraform v1.0.0 support (w/ fixes and CI testing) #95

Open
wants to merge 11 commits into
base: master
Choose a base branch
from

Conversation

gruberdev
Copy link

@gruberdev gruberdev commented Jun 9, 2021

Fixes:

Changelog

  • Added python-hcl2, support for HCL 2
  • Added support for Terraform v1.0.0 and backwards compatibility
  • Added a Terratest-capable feature to be CI-ingrated so its dockerfile build can be test automatically

Since the maintainer has been AFK for quite some time, I will also create a separate Docker container for those who want to run it without having to deal with the hassle for building and applying the fixes. Hang on. My repository can be found here.

ImgBotApp and others added 5 commits May 18, 2021 02:57
*Total -- 252.61kb -> 197.53kb (21.8%)

/doc/blastradius-interactive.png -- 135.81kb -> 97.29kb (28.36%)
/blastradius/server/static/example/demo-1/demo-1.svg -- 11.62kb -> 9.25kb (20.41%)
/blastradius/server/static/example/demo-2/demo-2.svg -- 25.76kb -> 20.94kb (18.7%)
/blastradius/server/static/example/demo-3/demo-3.svg -- 21.07kb -> 17.22kb (18.26%)
/doc/blast-radius-demo.svg -- 58.35kb -> 52.82kb (9.47%)

Signed-off-by: ImgBotApp <[email protected]>
@gruberdev gruberdev changed the title Terraform v1.0.0 support (w/ fixes) Terraform v1.0.0 support (w/ fixes and CI testing) Jun 9, 2021
@gruberdev
Copy link
Author

gruberdev commented Aug 11, 2021

Integrated changes from:

@gerrywastaken
Copy link

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

@gerrywastaken gerrywastaken mentioned this pull request Aug 19, 2021
@gruberdev
Copy link
Author

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

I will do that eventually, just need to find some spare time on my routine.

@sfc-gh-jlove
Copy link

@gruberdev awesome work!
I recommend splitting off (while still giving credit to the original) so you are no longer just a fork of this repository. This way yoru fork is more discoverable, can track issues and has the search feature on github. Otherwise it just appears as one of the many forks (in varying states) of this project. It would be awesome if you updated the Readme so that it mentions your docker image.

I will do that eventually, just need to find some spare time on my routine.

@gruberdev did you ever find the spare time?

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