Skip to content

Conversation

@sshota0809
Copy link
Contributor

Q A
πŸ› Bug fix? no
πŸš€ New feature? no
⚠ Deprecations? no
❌ BC Break no
πŸ”— Related issues #119
❓ Documentation no

Description

I'm done with replacing manual terraform execution with hashicorp/terraform-exec
Please review my commits!

related to snyk#119

Signed-off-by: sshota0809 <8736380+sshota0809@users.noreply.github.com>
related to snyk#119

Signed-off-by: sshota0809 <8736380+sshota0809@users.noreply.github.com>
@sshota0809 sshota0809 requested a review from a team as a code owner February 5, 2021 12:37
related to snyk#119

Signed-off-by: sshota0809 <8736380+sshota0809@users.noreply.github.com>
@codecov
Copy link

codecov bot commented Feb 5, 2021

Codecov Report

Merging #209 (528c414) into main (4eda65b) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #209   +/-   ##
=======================================
  Coverage   69.81%   69.81%           
=======================================
  Files         211      211           
  Lines        4740     4740           
=======================================
  Hits         3309     3309           
  Misses       1166     1166           
  Partials      265      265           

Copy link
Contributor

@eliecharra eliecharra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That look awesome, nothing to say πŸ‘πŸ»

I tried locally and it seems that terraform apply errors are displayed twice, could you investigate ?

image

@eliecharra eliecharra added the kind/maintenance Refactoring or changes to the workspace label Feb 5, 2021
@eliecharra
Copy link
Contributor

Hmm it seems to be an issue with my IDE (Goland), when running test with make acc it seems I don't have any issues

image

@sshota0809
Copy link
Contributor Author

@eliecharra
It seems wired. Should I investigate about it more?

@eliecharra
Copy link
Contributor

@eliecharra
It seems wired. Should I investigate about it more?

let me rather investigate this on my side.

I'll keep you informed

@eliecharra eliecharra merged commit 938a336 into snyk:main Feb 5, 2021
@eliecharra eliecharra linked an issue Feb 8, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

kind/maintenance Refactoring or changes to the workspace

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Replace manual terraform execution and use hashicorp provided library

2 participants