Skip to content

ci: fix Detect Changes job failing on push to main#26717

Open
yan-3005 wants to merge 1 commit intomainfrom
ram/fix-unit-tests-action
Open

ci: fix Detect Changes job failing on push to main#26717
yan-3005 wants to merge 1 commit intomainfrom
ram/fix-unit-tests-action

Conversation

@yan-3005
Copy link
Contributor

Add checkout step before dorny/paths-filter@v3 in the changes job. For push events, paths-filter runs git branch --show-current locally which fails without a checkout; pull_request events use the GitHub API and are unaffected.

Describe your changes:

Fixes

Type of change:

  • Bug fix
  • Improvement
  • New feature
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation

Checklist:

  • I have read the CONTRIBUTING document.
  • My PR title is Fixes <issue-number>: <short explanation>
  • I have commented on my code, particularly in hard-to-understand areas.
  • For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

Add checkout step before dorny/paths-filter@v3 in the changes job.
For push events, paths-filter runs git branch --show-current locally
which fails without a checkout; pull_request events use the GitHub API
and are unaffected.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@yan-3005 yan-3005 self-assigned this Mar 24, 2026
@yan-3005 yan-3005 added safe to test Add this label to run secure Github workflows on PRs backend labels Mar 24, 2026
Copilot AI review requested due to automatic review settings March 24, 2026 07:52
@gitar-bot
Copy link

gitar-bot bot commented Mar 24, 2026

Code Review ✅ Approved

Fixes the Detect Changes job that was failing on push to main by updating the CI configuration. No issues found.

Options

Auto-apply is off → Gitar will not commit updates to this branch.
Display: compact → Showing less information.

Comment with these commands to change:

Auto-apply Compact
gitar auto-apply:on         
gitar display:verbose         

Was this helpful? React with 👍 / 👎 | Gitar

Copy link
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

Fixes the CI workflow’s “Detect Changes” job on push to main by ensuring the repository is checked out before running dorny/paths-filter@v3, which relies on local git state for non-PR events.

Changes:

  • Added an actions/checkout@v4 step to the changes job (skipped for workflow_dispatch).
  • Ensures dorny/paths-filter@v3 can run successfully on push events by having a local git repository available.

@github-actions
Copy link
Contributor

OpenMetadata Service New-Code Coverage

✅ No changed production Java files under openmetadata-service/src/main/java. Coverage gate skipped.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backend safe to test Add this label to run secure Github workflows on PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants