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

chore: repository cleanup #19

Merged
merged 7 commits into from
Feb 28, 2023
Merged

chore: repository cleanup #19

merged 7 commits into from
Feb 28, 2023

Conversation

chris3ware
Copy link
Member

pre-commit hooks have been replaced with trunk and GitHub Actions.

Trunk overlapped with pre-commit with regards to terraform formatting and linting; to simplify things trunk was chosen as the single tool to use.

@chris3ware chris3ware requested a review from a team as a code owner February 27, 2023 15:53
The standard block delimeters differ to those used my pre-commit, so they hade to be changed to work with the GitHub Action
These files have been renamed because the `--tfvars-file` argument
should be based to the tfsec-pr workflow. For now this is much simpler
if they are all names the same.
These files should be included now, due to the rename in the previous
commit - with the exception of the vpc directory.
@chris3ware chris3ware merged commit 522d716 into main Feb 28, 2023
@chris3ware chris3ware deleted the chore-repo-man branch February 28, 2023 13:18
chris3ware added a commit that referenced this pull request Feb 28, 2023
pre-commit hooks have been replaced with trunk and GitHub Actions. 

Trunk overlapped with pre-commit with regards to terraform formatting
and linting; to simplify things trunk was chosen as the single tool to
use.
chris3ware added a commit that referenced this pull request Feb 28, 2023
pre-commit hooks have been replaced with trunk and GitHub Actions. 

Trunk overlapped with pre-commit with regards to terraform formatting
and linting; to simplify things trunk was chosen as the single tool to
use.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

1 participant