Skip to content

feat(ci): migrate php checks , docs, and php unit tests from Drone to GitHub Actions#294

Open
Ashim-Stha wants to merge 1 commit intomainfrom
migrate-to-github-actions
Open

feat(ci): migrate php checks , docs, and php unit tests from Drone to GitHub Actions#294
Ashim-Stha wants to merge 1 commit intomainfrom
migrate-to-github-actions

Conversation

@Ashim-Stha
Copy link
Copy Markdown
Contributor

@Ashim-Stha Ashim-Stha commented Mar 26, 2026

Description

Migrates the following Drone CI pipelines to GitHub Actions:

  • php-checks - codestyle, phpstan, phan
  • unit-tests-without-coverage
  • unit-tests-with-coverage
  • docs

Related Issue

@Ashim-Stha Ashim-Stha self-assigned this Mar 26, 2026
@Ashim-Stha Ashim-Stha force-pushed the migrate-to-github-actions branch 7 times, most recently from 3cb37e3 to 60883e6 Compare March 27, 2026 09:06
@Ashim-Stha Ashim-Stha changed the title feat: ci, migrate to github actions feat(ci): migrate php checks and php unit tests from Drone to GitHub Actions Mar 27, 2026
@Ashim-Stha Ashim-Stha force-pushed the migrate-to-github-actions branch 6 times, most recently from 82d6dc7 to f59b2fe Compare March 30, 2026 04:59
@Ashim-Stha Ashim-Stha changed the title feat(ci): migrate php checks and php unit tests from Drone to GitHub Actions feat(ci): migrate php checks , docs, and php unit tests from Drone to GitHub Actions Mar 30, 2026
@Ashim-Stha Ashim-Stha changed the title feat(ci): migrate php checks , docs, and php unit tests from Drone to GitHub Actions feat(ci): migrate php checks , docs, and php unit tests from Drone to GitHub Actions Mar 30, 2026
@Ashim-Stha Ashim-Stha mentioned this pull request Mar 30, 2026
8 tasks
@Ashim-Stha Ashim-Stha force-pushed the migrate-to-github-actions branch from f59b2fe to 626dc14 Compare March 30, 2026 06:36
@Ashim-Stha Ashim-Stha marked this pull request as ready for review March 30, 2026 06:39
@Ashim-Stha Ashim-Stha force-pushed the migrate-to-github-actions branch from 626dc14 to 7f6475c Compare March 30, 2026 10:59
…GitHub Actions

Signed-off-by: Ashim Shrestha <ashimshrestha2384@gmail.com>
@Ashim-Stha Ashim-Stha force-pushed the migrate-to-github-actions branch from 7f6475c to 4cf2b26 Compare March 30, 2026 11:11
Copy link
Copy Markdown
Member

@saw-jan saw-jan left a comment

Choose a reason for hiding this comment

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

Referencing:

✅ Allowed Actions Policy

Only the following GitHub Actions are allowed:

  • Official GitHub Actions
  • Actions from verified publishers

Any exceptions must be discussed with the development team before use.

we need to confirm (with dev team) that we are okay to use these actions

steps:
- uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2

- uses: shivammathur/setup-php@accd6127cb78bee3e8082180cb391013d204ef9f # v2.37.0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

most commonly used php setup but not verified.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

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.

See https://github.com/owncloud/reusable-workflows/blob/main/.github/workflows/php-unit.yml
and any other workflows in that folder. Those are being used with ownCloud core and apps.
Use anything from there as examples.

I don't think that we will be using https://github.com/owncloud-ci/php because actually we are easily using shivammathur/setup-php to get and configure PHP.

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.

github.com/owncloud/impersonate/pull/256 has GitHub workflow actions that I am doing for ownCloud core and apps. So look there also to get any ideas.

- run: composer install

- name: docs generate
uses: docker://phpdoc/phpdoc:3
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

using docker image. needs to clarify if this is ok.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

uses: docker://phpdoc/phpdoc:3

- name: publish api docs
uses: peaceiris/actions-gh-pages@4f9cc6602d3f66b9c108549d475ec49e8ef4d45e # v4.0.0
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

from unverified creator

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@Ashim-Stha
Copy link
Copy Markdown
Contributor Author

what will happen to owncloud-ci images? we are using ubuntu and php images from marketplace
cc: @phil-davis , @DeepDiver1975

@Ashim-Stha Ashim-Stha requested a review from phil-davis March 30, 2026 11:26
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.

3 participants