Skip to content

feat(ci): add Playwright testing workflow#62

Open
ccailly wants to merge 5 commits into
glpi-project:v1from
ccailly:feature/implement-playwright
Open

feat(ci): add Playwright testing workflow#62
ccailly wants to merge 5 commits into
glpi-project:v1from
ccailly:feature/implement-playwright

Conversation

@ccailly

@ccailly ccailly commented Jun 8, 2026

Copy link
Copy Markdown
Member

Add support for automatically detecting and running Playwright end-to-end (E2E)

@ccailly ccailly requested review from Rom1-B and stonebuzz and removed request for Rom1-B June 8, 2026 12:08
@ccailly ccailly self-assigned this Jun 8, 2026
@cedric-anne cedric-anne requested a review from froozeify June 8, 2026 12:21

@froozeify froozeify left a comment

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.

I saw a few more hard-coded 8.5 matching the php version I'll propose to use the input.php-version

Comment thread .github/workflows/continuous-integration.yml Outdated
Comment thread .github/workflows/continuous-integration.yml
@Rom1-B Rom1-B self-requested a review June 9, 2026 05:57

@Rom1-B Rom1-B 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.

Do you have a plugin ready to test these changes with Act before merging?

Comment thread .github/workflows/continuous-integration.yml Outdated
Comment thread .github/workflows/continuous-integration.yml Outdated
Comment thread .github/workflows/continuous-integration.yml Outdated
Comment thread .github/workflows/continuous-integration.yml
Comment thread .github/workflows/continuous-integration.yml
@ccailly ccailly marked this pull request as draft June 9, 2026 07:08
@ccailly ccailly force-pushed the feature/implement-playwright branch 3 times, most recently from d22beba to 78ff99b Compare June 9, 2026 07:56
@ccailly ccailly force-pushed the feature/implement-playwright branch from 78ff99b to 4e592d9 Compare June 9, 2026 08:22
ccailly and others added 2 commits June 9, 2026 10:23
@ccailly ccailly force-pushed the feature/implement-playwright branch from 0c5f429 to 9f51ea6 Compare June 9, 2026 08:33
@ccailly

ccailly commented Jun 9, 2026

Copy link
Copy Markdown
Member Author

Do you have a plugin ready to test these changes with Act before merging?

The workflows plugin includes Playwright tests; you can see the CI in action on this Action: https://github.com/glpi-network/workflows/actions/runs/27196256448

@ccailly ccailly requested review from Rom1-B and froozeify June 9, 2026 09:18
@ccailly ccailly marked this pull request as ready for review June 9, 2026 09:21
Comment thread .github/workflows/continuous-integration.yml Outdated
Comment thread .github/workflows/continuous-integration.yml
@ccailly ccailly requested a review from Rom1-B June 9, 2026 13:43
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