defenseunicorns / delivery-aws-iac

Apache License 2.0
16 stars 5 forks source link

Update automated workflows based on new ADRs #323

Closed RothAndrew closed 1 year ago

RothAndrew commented 1 year ago

Some of the stuff here won't take effect until it is in main. I did my testing over in this repo: https://github.com/RothAndrew/tmp-renovate-test

Breakdown of changes:

BEGIN_COMMIT_OVERRIDE refactor(automation): refactor the update-command workflow so that the bulk of the logic is in a reusable action, since it will now be used in 2 different places (#323)

refactor(automation): simplify the if statements now that we aren't trying to run tests on push to main anymore (#323)

refactor(automation): refactor parse logic to make it a bit simpler (#323)

ci(pr-automation): delete the auto-labeling workflow (#323)

ci(main): delete the pre-commit-trunk workflow now that we aren't trying to run it on commits to main (#323)

refactor(pr-automation): simplify the GitHub context name that the E2E tests use so that we can require them regardless of how they were triggered (#323)

ci(test): update the auto-test workflow to reflect the decisions documented in ADR 8 (#323)

END_COMMIT_OVERRIDE

zack-is-cool commented 1 year ago

can you give a super high level overview on what changes are happening in this pr?

RothAndrew commented 1 year ago

Lol it's like I was reading your mind or something, I just finished writing it. See the PR description 😄