Skip to content

Do not assume target branch is on the same fork as the CI#424

Merged
holly-cummins merged 1 commit intoquarkusio:mainfrom
holly-cummins:fix-error-in-sync-workflow
Apr 9, 2026
Merged

Do not assume target branch is on the same fork as the CI#424
holly-cummins merged 1 commit intoquarkusio:mainfrom
holly-cummins:fix-error-in-sync-workflow

Conversation

@holly-cummins
Copy link
Copy Markdown
Collaborator

Should fix

fatal: ambiguous argument 'origin/ootb..origin/fix_415': unknown revision or path not in the working tree.

happening in the PR workflow for #422.

@holly-cummins holly-cummins force-pushed the fix-error-in-sync-workflow branch from abf8906 to 35a8589 Compare April 9, 2026 19:09
@holly-cummins holly-cummins requested a review from edeandrea April 9, 2026 19:10
@edeandrea
Copy link
Copy Markdown
Collaborator

Once this thing works correctly, can it PR itself to the other branch? :)

the automation has to exist on the ootb branch too for PRs targeting that branch to pick it up

@holly-cummins
Copy link
Copy Markdown
Collaborator Author

Once this thing works correctly, can it PR itself to the other branch? :)

:)

the automation has to exist on the ootb branch too for PRs targeting that branch to pick it up

I had a think about that, and decided it didn't. But maybe I've got that wrong. It might need some experimentation.

@holly-cummins holly-cummins force-pushed the fix-error-in-sync-workflow branch from 35a8589 to 8b8b4b6 Compare April 9, 2026 20:10
@holly-cummins holly-cummins merged commit aed6681 into quarkusio:main Apr 9, 2026
23 checks passed
@edeandrea
Copy link
Copy Markdown
Collaborator

In order for an automation to run on a PR branch it has to exist on the branch that the PR is targeting. That's why there is a separate .github/workflows/main.yml on the ootb branch.

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