Skip to content

Conversation

@Grigoriy0
Copy link

@Grigoriy0 Grigoriy0 commented Sep 6, 2023

Added Dockerfile

Type of Change

  • Bug Fix
  • New Feature
  • Refactor
  • Documentation

Checklist

All Submissions:

  • Have you checked to ensure there aren't other open Pull Requests for the same update/change?
  • Have you written Documentation/Tests?
  • Have you read done your own code-review?

Changes to Core Features:

  • Have you added an explanation of what your changes do and why you'd like us to include them?
  • Have you written new tests for your core changes, as applicable?
  • Have you successfully run tests with your changes locally?

@patrickchugh
Copy link
Owner

Thanks this is awesome, please wait for the next merge as I am rebuilding the interpreter engine to use the Terraform binary now to parse the code in the first pass. Please test against that once I merge that branch and then we can include this.

@patrickchugh
Copy link
Owner

@Grigoriy0 Hello, thank you for your feedback and thanks for testing out Terravision and helping to make this a better product. I have recently completed a major revamp to the code which uses the terraform binary to download source files and generate initial relationships, before augmenting it with source code parsing as before. This is a significant departure from the previous approach and whilst slower, ensures a more accurate output and will handle all terraform in built functions now when encountered in the source. Please do a git pull and re-test again your PR with the latest source code. If needed, amend and I will happily merge it. Once again, thanks for helping and look forward to hearing from you soon.

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.

2 participants