Skip to content

fix(commonalities-sync): checkout tooling repo explicitly in reusable workflow - #459

Merged
hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/commonalities-sync-checkout-repo
Sep 19, 2026
Merged

hdamker merged 1 commit into
camaraproject:mainfrom
hdamker:fix/commonalities-sync-checkout-repo

Conversation

@hdamker

@hdamker hdamker commented Sep 19, 2026

Copy link
Copy Markdown
Contributor

What type of PR is this?

bug

What this PR does / why we need it:

Both "Checkout tooling" steps in commonalities-regression-sync.yml omitted repository:, so actions/checkout defaulted to the calling repo instead of camaraproject/tooling, where validation/scripts and requirements.txt actually live. This breaks the pip install -r requirements.txt step whenever the workflow is called from camaraproject/Commonalities (the checkout resolves to Commonalities, which has no requirements.txt).

Adds repository: camaraproject/tooling to both checkout steps.

Which issue(s) this PR fixes:

Fixes #

Special notes for reviewers:

Observed failure: https://github.com/camaraproject/Commonalities/actions/runs/35433039479/job/105870873151#step:5:16

Changelog input

 release-note
Fix CommonalitiesTest regression sync workflow checking out the wrong repository for its script dependencies.

Additional documentation

This section can be blank.

docs

… workflow

Both "Checkout tooling" steps omitted `repository:`, so actions/checkout
defaulted to the calling repo (Commonalities) instead of tooling, where
validation/scripts and requirements.txt actually live.
@hdamker
hdamker merged commit e6ba792 into camaraproject:main Sep 19, 2026
8 checks passed
@hdamker
hdamker deleted the fix/commonalities-sync-checkout-repo branch September 19, 2026 09:06
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