Skip to content

ci: add workflow trigger when pushing to main#87

Open
ViTeXFTW wants to merge 1 commit intodevfrom
ci/run-test-on-main
Open

ci: add workflow trigger when pushing to main#87
ViTeXFTW wants to merge 1 commit intodevfrom
ci/run-test-on-main

Conversation

@ViTeXFTW
Copy link
Owner

@ViTeXFTW ViTeXFTW commented Feb 20, 2026

Description

Trigger test workflow when pushing to main to ensure tests pass on main

Type of change

  • Bugfix
  • Feature
  • Refactoring
  • Formatting
  • Build
  • CI
  • Documentation
  • Tests
  • Other

@ViTeXFTW
Copy link
Owner Author

@greptileai

@greptile-apps
Copy link

greptile-apps bot commented Feb 20, 2026

Greptile Summary

Added push trigger for main branch to the PR test workflow, ensuring tests run when code is pushed directly to main (e.g., emergency hotfixes).

  • Simple and straightforward CI configuration change
  • Workflow will now run tests on direct pushes to main, in addition to existing PR triggers
  • Minor consideration: PRs targeting main will trigger this workflow twice (once on PR open/update, once on merge), though this redundancy is typically not problematic

Confidence Score: 4/5

  • Safe to merge with minor workflow optimization opportunity
  • The change is simple and achieves its stated goal of running tests on pushes to main. The only minor consideration is potential duplicate workflow runs when PRs are merged to main, but this doesn't affect functionality or safety.
  • No files require special attention

Important Files Changed

Filename Overview
.github/workflows/pr-test.yml Added push trigger for main branch to ensure tests run when code is pushed directly to main

Last reviewed commit: 0e5e757

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

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

1 file reviewed, 1 comment

Edit Code Review Agent Settings | Greptile

Comment on lines +9 to +11
push:
branches:
- main
Copy link

Choose a reason for hiding this comment

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

When a PR is merged to main, this workflow will run twice: once from the pull_request trigger and once from the push trigger. Consider adding if: github.event_name != 'pull_request' to the job or removing main from the pull_request.branches list to avoid redundant runs.

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.

1 participant