Skip to content

[5.3] Add CI Github workflow #45602

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 32 commits into from
Jun 25, 2025
Merged

[5.3] Add CI Github workflow #45602

merged 32 commits into from
Jun 25, 2025

Conversation

laoneo
Copy link
Member

@laoneo laoneo commented Jun 12, 2025

Summary of Changes

Adds a Github workflow file which does the same code style checks and testing as we do now in drone, but with chrome and edge browsers. Firefox is currently not supported. An additional parameter is added to the entrypoint file to define the browser to run.

@laoneo laoneo changed the title Ghactions [5.3] Add CI Github workflow Jun 12, 2025
@brianteeman
Copy link
Contributor

nice work

@Hackwar
Copy link
Member

Hackwar commented Jun 20, 2025

Why are you running the system tests on chrome and not Firefox like right now?

@laoneo
Copy link
Member Author

laoneo commented Jun 20, 2025

On GH is only chrome available. Can't find the issue right now, but firefox was not working.

laoneo and others added 3 commits June 20, 2025 20:12
@Hackwar
Copy link
Member

Hackwar commented Jun 20, 2025

Since we are using our own docker image for the cypress tests, firefox should still be available.

@laoneo
Copy link
Member Author

laoneo commented Jun 21, 2025

The install task of cypress adds the browsers, they are not in the image.

@rdeutz rdeutz merged commit f62a9c5 into 5.3-dev Jun 25, 2025
61 of 63 checks passed
@rdeutz
Copy link
Contributor

rdeutz commented Jun 25, 2025

Thanks!

@laoneo laoneo deleted the ghactions branch June 25, 2025 06:06
@laoneo laoneo added this to the Joomla! 5.3.2 milestone Jun 25, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants