Skip to content

1604 automated ci workflows should run on push to main#1642

Merged
SFJohnson24 merged 11 commits intomainfrom
1604-automated-ci-workflows-should-run-on-push-to-main
Mar 2, 2026
Merged

1604 automated ci workflows should run on push to main#1642
SFJohnson24 merged 11 commits intomainfrom
1604-automated-ci-workflows-should-run-on-push-to-main

Conversation

@gerrycampion
Copy link
Collaborator

@gerrycampion gerrycampion commented Feb 27, 2026

  • Renamed files to use a consistent hyphen convention, instead of underscores
  • Removed automated-ci because it is redundant and prevents granularity
  • Updated all used actions to their latest version
  • renamed rule-tester to deploy-rule-tester to make its functionality less ambiguous
  • reverted deploy-rule-tester to run on all pull request events, now that we removed the auto merge update. test-rule-editor-preview will run after deploy-rule-tester is triggered from a pull-request or push
  • removed inputs from test-rule-editor-preview so that workflow_call will always use github.ref
  • updated lint-format to also run on push to main and workflow_dispatch. Replaced our step with the changed-files action, which automatically handles determining comparison commits for for workflow_dispatch, pull_request, and pushes
  • updated test-unit to also run on push to main
  • updated test-suite to also run on push to main and removed unused workflow_call
  • Clarified description for Forked repo pr number selection

@gerrycampion gerrycampion linked an issue Feb 27, 2026 that may be closed by this pull request
@gerrycampion gerrycampion marked this pull request as ready for review February 27, 2026 19:50
Copy link
Collaborator

@SFJohnson24 SFJohnson24 left a comment

Choose a reason for hiding this comment

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

this PR correctly addresses the bullets laid out in the PR--mainly syntax, version updating for the git action, some tweaks to workflow on conditions, and implementing changed-files for linting.

@SFJohnson24 SFJohnson24 merged commit bfe2e93 into main Mar 2, 2026
12 checks passed
@SFJohnson24 SFJohnson24 deleted the 1604-automated-ci-workflows-should-run-on-push-to-main branch March 2, 2026 18:04
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.

automated-ci workflows should run on push to main

2 participants