Skip to content

ci: migrate docs and e2e to shared workflows - #113

Merged
CybotTM merged 1 commit into
mainfrom
ci/use-shared-workflows
Mar 16, 2026
Merged

ci: migrate docs and e2e to shared workflows#113
CybotTM merged 1 commit into
mainfrom
ci/use-shared-workflows

Conversation

@CybotTM

@CybotTM CybotTM commented Mar 16, 2026

Copy link
Copy Markdown
Member

Summary

  • Replace local docs.yml (render-guides + warning check + artifact upload) with a thin caller to netresearch/typo3-ci-workflows/.github/workflows/docs.yml@main, matching the pattern used by t3x-contexts.
  • Replace local e2e.yml (Playwright + MariaDB + PHP server + TYPO3 setup) with a thin caller to netresearch/typo3-ci-workflows/.github/workflows/e2e.yml@main.

Note: The shared e2e.yml workflow is being created in a parallel PR at netresearch/typo3-ci-workflows. The caller references @main and will work once that PR is merged.

Test plan

  • Verify docs workflow triggers on Documentation/** changes after merge
  • Verify e2e workflow runs successfully after the shared e2e.yml is merged upstream

Copilot AI review requested due to automatic review settings March 16, 2026 11:49
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a summary for this pull request due to the file types involved not being currently supported.

@github-actions

Copy link
Copy Markdown
Contributor

Dependency Review

The following issues were found:
  • ✅ 0 vulnerable package(s)
  • ✅ 0 package(s) with incompatible licenses
  • ✅ 0 package(s) with invalid SPDX license definitions
  • ⚠️ 1 package(s) with unknown licenses.
See the Details below.

License Issues

.github/workflows/e2e.yml

PackageVersionLicenseIssue Type
netresearch/typo3-ci-workflows/.github/workflows/e2e.ymlmainNullUnknown License

OpenSSF Scorecard

PackageVersionScoreDetails
actions/netresearch/typo3-ci-workflows/.github/workflows/e2e.yml main UnknownUnknown

Scanned Files

  • .github/workflows/e2e.yml

@github-actions github-actions Bot added the ci CI/CD changes label Mar 16, 2026

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

Migrates the repository’s docs and E2E GitHub Actions workflows to thin “caller” workflows that delegate execution to shared workflows in netresearch/typo3-ci-workflows.

Changes:

  • Replaces the local Playwright/MariaDB/PHP-based E2E workflow with a reusable-workflow caller.
  • Replaces the local documentation render + warnings + artifact workflow with a reusable-workflow caller.
  • Adjusts docs workflow triggers to also run when the workflow file itself changes.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
.github/workflows/e2e.yml Swaps the in-repo E2E implementation for a reusable workflow call with inputs (PHP version, DB image, artifact paths).
.github/workflows/docs.yml Swaps the in-repo docs build implementation for a reusable workflow call and expands trigger paths.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread .github/workflows/e2e.yml
Comment thread .github/workflows/docs.yml

@github-actions github-actions Bot 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.

Automated approval for solo maintainer project

This PR has passed all automated quality gates:

  • ✅ Static analysis (PHPStan)
  • ✅ Code style (PHP-CS-Fixer)
  • ✅ Unit & functional tests
  • ✅ Security scanning
  • ✅ Dependency review

See SECURITY_CONTROLS.md for compensating controls documentation.

@CybotTM
CybotTM enabled auto-merge March 16, 2026 12:06
@CybotTM
CybotTM merged commit cebcf12 into main Mar 16, 2026
47 checks passed
@CybotTM
CybotTM deleted the ci/use-shared-workflows branch March 16, 2026 12:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci CI/CD changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants