Convert to `context.tf` #5

Nuru · 2020-08-28T21:16:43Z

what

Convert to context.tf
Implement enabled flag
Enhance example
Updated lint.yml to remove superlinter and use latest version of build-harness for readme linting.

Standardize
Demonstrate how to avoid creating resources when not enabled
Latest version of build-harness is used to build README and same version needs to be used to lint it.
Superlinter is not ready for production. There is a bug in the go linting and problems with terrascan. This can be added back once the project gets a bit more mature.

osterman · 2020-08-28T21:31:48Z

Would it be quick to also update it to go modules?

Nuru · 2020-08-28T22:38:36Z

/test all

Nuru · 2020-08-28T23:24:46Z

/test all

Convert to context.tf

e21c71c

Nuru requested a review from a team as a code owner August 28, 2020 21:16

Nuru requested review from Makeshift, nitrocode, aknysh, goruha, htplbc, maximmi, osterman and danjbh August 28, 2020 21:16

htplbc previously approved these changes Aug 28, 2020

View reviewed changes

Convert to go modules, further enhance tests

f3739c1

Nuru dismissed htplbc’s stale review via f3739c1 August 28, 2020 22:37

Nuru requested a review from a team as a code owner August 28, 2020 22:37

Nuru added 2 commits August 28, 2020 15:41

fmt and README

85d6b53

Update lint.yml workflow, disable superlinter

7638c38

aknysh approved these changes Aug 30, 2020

View reviewed changes

Nuru merged commit 3a6464e into master Aug 30, 2020

Nuru deleted the context branch August 30, 2020 01:14