Skip to content

ci: run CI workflow on push to main#27392

Merged
pelikhan merged 1 commit intomainfrom
copilot/run-ci-on-push-to-main
Apr 20, 2026
Merged

ci: run CI workflow on push to main#27392
pelikhan merged 1 commit intomainfrom
copilot/run-ci-on-push-to-main

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 20, 2026

  • Inspect existing /home/runner/work/gh-aw/gh-aw/.github/workflows/ci.yml triggers
  • Run baseline validation (make lint build test-unit) before changes
  • Add push trigger for main to CI workflow
  • Re-run targeted workflow validation after the change (make actionlint)
  • Run required pre-commit validation (make agent-finish) (fails on pre-existing gosec findings unrelated to this change)
  • Commit and push changes
  • Run parallel validation on PR changes (Code Review passed; CodeQL timed out per tool and should not be re-run)
  • Create/update PR with final summary

Agent-Logs-Url: https://github.com/github/gh-aw/sessions/457937e8-a102-4d4b-a3c9-213bda26dc90

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI requested a review from pelikhan April 20, 2026 16:35
@pelikhan pelikhan marked this pull request as ready for review April 20, 2026 16:37
Copilot AI review requested due to automatic review settings April 20, 2026 16:37
@pelikhan pelikhan merged commit 8b16d53 into main Apr 20, 2026
@pelikhan pelikhan deleted the copilot/run-ci-on-push-to-main branch April 20, 2026 16:37
@github-actions github-actions bot mentioned this pull request Apr 20, 2026
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the CI GitHub Actions workflow so it runs automatically when changes are pushed to the main branch (in addition to existing scheduled/manual triggers).

Changes:

  • Add push trigger for main to .github/workflows/ci.yml.
Show a summary per file
File Description
.github/workflows/ci.yml Adds a push trigger for main so CI runs on merges/pushes to the default branch.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 1/1 changed files
  • Comments generated: 1

Comment thread .github/workflows/ci.yml
branches:
- main
schedule:
- cron: 0 * * * *
Copy link

Copilot AI Apr 20, 2026

Choose a reason for hiding this comment

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

The schedule trigger’s list item formatting here is inconsistent with the rest of the repo and harder to read (e.g. .github/workflows/docs.yml:4-8 indents the - cron under schedule and quotes the cron string). Consider updating this block to match that style (indent - cron under schedule and quote the cron expression).

Suggested change
- cron: 0 * * * *
- cron: "0 * * * *"

Copilot uses AI. Check for mistakes.
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.

3 participants