Skip to content

Fix publish workflows fork PR lookup and docs trigger - #258

Merged
mariuswbr merged 2 commits into
eclipse-score:mainfrom
etas-contrib:fix-fork-pr-lookup
Oct 21, 2026
Merged

mariuswbr merged 2 commits into
eclipse-score:mainfrom
etas-contrib:fix-fork-pr-lookup

Conversation

@ryan-steel

@ryan-steel ryan-steel commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

Bugfix

Description

Docs and quality pack publishing actions are failing on branches made from company-contrib forks when the forks are renamed e.g. etas-contrib/score_time. See docs-publish and quality_pack-publish failures.

Fix docs-publish by upgrading the pinned commit hash to post-fix.
Fix quality pack publish by adding the same fix added to the docs-publish workflow in cicd-workflows.

Also changes doc workflow to work on pull_request rather than pull_request_target so that the workflow builds for the branch that the PR is opened for rather than the target branch (main).

Related ticket

closes #252

@github-actions

github-actions Bot commented Oct 20, 2026 •

Copy link
Copy Markdown
Contributor

License Check Results

🚀 The license check job ran with the Bazel command:

bazel run //:license-check

Status: ⚠️ Needs Review

Click to expand output
[License Check Output]
Extracting Bazel installation...
Starting local Bazel server (8.6.0) and connecting to it...
INFO: Invocation ID: 03eac773-3b5b-45b3-b678-25d613417fac
Computing main repo mapping: 
Computing main repo mapping: 
WARNING: For repository 'platforms', the root module requires module version platforms@1.0.0, but got platforms@1.1.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_tooling', the root module requires module version score_tooling@2.2.1, but got score_tooling@2.2.2 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'rules_oci', the root module requires module version rules_oci@2.2.7, but got rules_oci@2.3.0 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
WARNING: For repository 'score_devcontainer', the root module requires module version score_devcontainer@1.8.0, but got score_devcontainer@1.11.1 in the resolved dependency graph. Please update the version in your MODULE.bazel or set --check_direct_dependencies=off
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Computing main repo mapping: 
Loading: 
Loading: 3 packages loaded
WARNING: Target pattern parsing failed.
ERROR: Skipping '//:license-check': no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/time/time/BUILD
ERROR: no such target '//:license-check': target 'license-check' not declared in package '' defined by /home/runner/work/time/time/BUILD
INFO: Elapsed time: 14.524s
INFO: 0 processes.
ERROR: Build did NOT complete successfully
ERROR: Build failed. Not running target

@ryan-steel
ryan-steel deployed to workflow-approval October 20, 2026 12:48 — with GitHub Actions Active
@ryan-steel
ryan-steel deployed to workflow-approval October 20, 2026 12:48 — with GitHub Actions Active
@ryan-steel ryan-steel changed the title Fix fork pr lookup Fix publish workflows fork PR lookup and docs trigger Oct 20, 2026

@LudwigWeise LudwigWeise left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@mariuswbr
mariuswbr merged commit 70b638b into eclipse-score:main Oct 21, 2026
27 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in TIM - Time FT Oct 21, 2026
@ryan-steel
ryan-steel deleted the fix-fork-pr-lookup branch October 21, 2026 13:34

This branch was successfully deployed

1 active deployment
workflow-approval — a324d23d Deployed Oct 20, 2026 by ryan-steel via qnx-build (x86_64-qnx) / approval #1014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

bug: docs workflow

3 participants