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

Bring Main Branch to Working Branch #64

Merged
merged 8 commits into from
Mar 8, 2021
Merged

Bring Main Branch to Working Branch #64

merged 8 commits into from
Mar 8, 2021

Conversation

Breanna-Stryker
Copy link
Contributor

Description

No review needed, recapturing current work so mimic on going script work in my branch

Please explain the changes you've made
N/A

Issue reference

N/A

The issue this PR will close: #[issue number]

N/A

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

N/A

  • [/ ] Code compiles or validates correctly
  • [/ ] BASH scripts have been validated using shellcheck
  • [/ ] All tests pass (manual and automated)
  • [/ ] The documentation is updated to cover any new or changed features
  • [/ ] Markdown files have been linted using the recommended linter. (See .vscode/extensions.json.)
  • [/ ] Relevant issues are linked to this PR

brooke-hamilton and others added 8 commits March 2, 2021 09:36
* Create pull_request_template.md

* updated issue template
* add a config folder to centralize generation
CONTRIBUTING.md
…ration (#60)

* remove enclave name, make location part of mlz config
…k, exit on any errors (#63)

* add apply_tf and destroy_tf

* reverse order on apply

* use vars

* add helpers and refactor terraform commands

* exit if any subshell should error

* exit on errors

* add readme, shellcheck files

* clarify config generation

* Update README.md

Co-authored-by: Glenn Musa <glennmusa@users.noreply.github.com>
@Breanna-Stryker Breanna-Stryker merged commit bb39e5b into ezdeploy Mar 8, 2021
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.

3 participants