Skip to content

fix: Restore deploy-versioned-pages actions - #179

Merged
AlexanderLanin merged 1 commit into
eclipse-score:mainfrom
elektrobit-contrib:fix-legacy-docs-build
Aug 3, 2026
Merged

fix: Restore deploy-versioned-pages actions#179
AlexanderLanin merged 1 commit into
eclipse-score:mainfrom
elektrobit-contrib:fix-legacy-docs-build

Conversation

@lurtz

@lurtz lurtz commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

The old docs workflow uses this action @main and now fails. Due the use of pull_request_target this makes even switching to the new docs workflow impossible. Thus restore it until all users are migrated.

Caused by #124

The old docs workflow uses this action @main and now fails. Due the use of `pull_request_target` this makes even switching to the new docs workflow impossible. Thus restore it until all users are migrated.

Caused by eclipse-score#124
@AlexanderLanin

Copy link
Copy Markdown
Member

temp for some hours / days / weeks, before we delete it again. must be on "main" for legacy workflows 😢

@AlexanderLanin
AlexanderLanin merged commit d5dfe33 into eclipse-score:main Aug 3, 2026
2 checks passed
@lurtz
lurtz deleted the fix-legacy-docs-build branch August 4, 2026 07:52
Chahult added a commit to Chahult/config_management that referenced this pull request Aug 10, 2026
eclipse-score/cicd-workflows#124 split docs.yml into build-only docs.yml +
docs-publish.yml, but per eclipse-score/cicd-workflows#179 this broke PR-head
checkout for pull_request_target events (upstream-acknowledged, still
unresolved). Revert docs.yml to the same pre-split commit already used by
baselibs/logging/communication's external deps, which builds and deploys
docs in one reusable workflow and correctly checks out the PR head via
github.event.pull_request.head.ref regardless of pull_request_target.

- docs.yml: pin to af347722c7ae3ed85518895c11268d96ac728f62 (baselibs' pin)
  and drop the now-unneeded manual refs/pull/<n>/head checkout hack.
- remove docs-publish.yml: deploy is handled internally by the pinned
  docs.yml's docs-deploy job, so no separate publish caller is needed.
- copyright.yml, format.yml, license_check.yml, docs-cleanup.yml: unrelated
  to this bug, pins left unchanged from the prior fixed-SHA pinning commit.
Chahult added a commit to Chahult/config_management that referenced this pull request Aug 10, 2026
eclipse-score/cicd-workflows#124 split docs.yml into build-only docs.yml +
docs-publish.yml, but per eclipse-score/cicd-workflows#179 this broke PR-head
checkout for pull_request_target events (upstream-acknowledged, still
unresolved). Revert docs.yml to the same pre-split commit already used by
baselibs/logging/communication's external deps, which builds and deploys
docs in one reusable workflow and correctly checks out the PR head via
github.event.pull_request.head.ref regardless of pull_request_target.

- docs.yml: pin to af347722c7ae3ed85518895c11268d96ac728f62 (baselibs' pin)
  and drop the now-unneeded manual refs/pull/<n>/head checkout hack.
- remove docs-publish.yml: deploy is handled internally by the pinned
  docs.yml's docs-deploy job, so no separate publish caller is needed.
- copyright.yml, format.yml, license_check.yml, docs-cleanup.yml: unrelated
  to this bug, pins left unchanged from the prior fixed-SHA pinning commit.
michaelsaborov pushed a commit to eclipse-score/config_management that referenced this pull request Aug 11, 2026
eclipse-score/cicd-workflows#124 split docs.yml into build-only docs.yml +
docs-publish.yml, but per eclipse-score/cicd-workflows#179 this broke PR-head
checkout for pull_request_target events (upstream-acknowledged, still
unresolved). Revert docs.yml to the same pre-split commit already used by
baselibs/logging/communication's external deps, which builds and deploys
docs in one reusable workflow and correctly checks out the PR head via
github.event.pull_request.head.ref regardless of pull_request_target.

- docs.yml: pin to af347722c7ae3ed85518895c11268d96ac728f62 (baselibs' pin).
- copyright.yml, format.yml, license_check.yml, docs-cleanup.yml: pin to a
  fixed commit SHA instead of floating @main so upstream changes to these
  reusable workflows cannot silently alter our CI behavior.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

2 participants