Skip to content
This repository has been archived by the owner on Aug 28, 2023. It is now read-only.

feat: GitHub Folder #2

Merged
merged 6 commits into from
Jun 29, 2022
Merged

feat: GitHub Folder #2

merged 6 commits into from
Jun 29, 2022

Conversation

HarryET
Copy link
Contributor

@HarryET HarryET commented Jun 28, 2022

Changes

  • Added .github and some basic workflows

@HarryET HarryET added enhancement New feature or request actions GitHub Actions CI/CD labels Jun 28, 2022
@HarryET HarryET requested review from a team, bazaah and ganchoradkov and removed request for a team and bazaah June 28, 2022 16:40
.gitignore Show resolved Hide resolved
@HarryET HarryET requested a review from ffrizzo June 29, 2022 10:27
@HarryET HarryET requested a review from ffrizzo June 29, 2022 10:38
- name: Ensure Formatting
run: |
terraform fmt -recursive -check
if [ $? -ne 0 ]; then
Copy link

Choose a reason for hiding this comment

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

We don't need this if when we use -check terraform CLI will automatically fail. So we can keep it simple only with line 20 and remove others

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Removed those lines

Remove unnecessary if
@HarryET HarryET requested a review from ffrizzo June 29, 2022 13:26
@HarryET HarryET merged commit 558ec33 into main Jun 29, 2022
@ffrizzo ffrizzo deleted the harry/feat/github branch June 29, 2022 13:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
actions GitHub Actions CI/CD enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants