Skip to content

ci: add synchronize trigger to semantic PR title validation workflow#18

Merged
Aaron ("AJ") Steers (aaronsteers) merged 1 commit intomainfrom
devin/1769542748-add-synchronize-trigger
Jan 27, 2026
Merged

ci: add synchronize trigger to semantic PR title validation workflow#18
Aaron ("AJ") Steers (aaronsteers) merged 1 commit intomainfrom
devin/1769542748-add-synchronize-trigger

Conversation

@aaronsteers
Copy link
Member

@aaronsteers Aaron ("AJ") Steers (aaronsteers) commented Jan 27, 2026

Summary

Adds synchronize to the list of event types that trigger the semantic PR title validation workflow. Without this trigger, the PR title check only runs when a PR is opened, edited, or marked ready for review - but not when new commits are pushed. This causes GitHub to show the check as "missing" or stale after pushing new commits.

Review & Testing Checklist for Human

  • Confirm this matches the desired behavior: PR title validation should re-run when new commits are pushed to refresh the check status

Notes

This change aligns the workflow with other Airbyte repositories (PyAirbyte, airbyte-python-cdk, connector-builder-mcp, etc.) that already have all four triggers configured: opened, edited, ready_for_review, and synchronize.

Requested by: Aaron ("AJ") Steers (@aaronsteers)
Link to Devin run: https://app.devin.ai/sessions/9c86b01fddb14dd0a5fd73a4f019c7b2


Open with Devin

@devin-ai-integration
Copy link
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link

🎉 Thanks for opening this pull request!

Your contribution is appreciated. Here are some helpful commands you can use:

Quick Commands

  • /autofix - Auto-format and fix linting issues (ruff format + ruff check --fix)
  • /lock - Update the uv.lock file with latest dependencies

Available Poe Tasks

You can run any of these tasks using the slash command: /poe <task-name>

Core Tasks

  • /poe test - Run all tests
  • /poe test-fast - Run tests with fast exit on first failure
  • /poe lint - Check code style and quality
  • /poe format - Format code with ruff
  • /poe deps - Check for unused and missing dependencies
  • /poe check - Run format check, linting, dependency check, and tests

Quick Fixes

  • /poe fix - Auto-format and fix linting issues
  • /poe clean - Clean up build artifacts and cache

Build & Install

  • /poe build - Build the package
  • /poe install - Install with development dependencies

Other Commands

  • /poe version - Show package version
  • /poe pre-commit - Run pre-commit style checks

The CI will automatically run tests when you push commits. Happy coding! 🚀

@aaronsteers Aaron ("AJ") Steers (aaronsteers) marked this pull request as ready for review January 27, 2026 19:44
Copilot AI review requested due to automatic review settings January 27, 2026 19:44
Copy link

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

This PR enhances the semantic PR title validation workflow by adding the synchronize trigger event. This ensures that PR title validation re-runs when new commits are pushed, preventing the check from appearing as "missing" or stale in GitHub's status display.

Changes:

  • Added synchronize event type to the semantic PR title validation workflow triggers

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

@aaronsteers Aaron ("AJ") Steers (aaronsteers) merged commit 01efc9f into main Jan 27, 2026
17 checks passed
@aaronsteers Aaron ("AJ") Steers (aaronsteers) deleted the devin/1769542748-add-synchronize-trigger branch January 27, 2026 19:44
Copy link
Contributor

@devin-ai-integration devin-ai-integration bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional flag.

Open in Devin Review

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.

2 participants