Skip to content

[5.4] Add Support for Github Codespaces#45950

Merged
richard67 merged 24 commits intojoomla:5.4-devfrom
alikon:backport-codespaces
Sep 12, 2025
Merged

[5.4] Add Support for Github Codespaces#45950
richard67 merged 24 commits intojoomla:5.4-devfrom
alikon:backport-codespaces

Conversation

@alikon
Copy link
Contributor

@alikon alikon commented Aug 21, 2025

Summary of Changes

Backport [6.0] Add Support for Github Codespaces #45719 to 5.4

Testing Instructions

see #45719 and manual joomla/Manual#496

Actual result BEFORE applying this Pull Request

N/A

Expected result AFTER applying this Pull Request

An easy way to test PRs on the cloud without setting everything up manually and on your local machine

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

@muhme
Copy link
Contributor

muhme commented Sep 12, 2025

I have tested this item ✅ successfully on d71883c

@alikon @mahmoudmagdy1-1 @laoneo ✅ Successfully tested fpr 5.4-dev with branch from PR on own fork:

  • Created Codespace
  • Logged in to Joomla administrator backend, enabled 'Debug System' and 'Log Almost Everything', played arround and checked the Joomla log files with 4LOGS plugin and in VSC
  • Installed German language packet as extension by file upload and configured as Administrator default
  • Installed module zitat-service.de from JED, configured and checked it is working
  • Running some Cypress tests with GUI manually
  • Running full System Tests headless (takes 24 minutes –  I think that leaving Xdebug enabled all the time reduces performance and is therefore not recommended from my point of view, also 33 of 148 failed - but the env is for sure not recommended to run all the System Tests)
  • Logged in to phpMyAdmin
  • ❌ Failed to install Patchtester 4.4.0 with file upload
    • Solved by installing via URL, fetching data works, applying a PR works and file is changed
  • ❌ Tried to register for automated updates: Error while registering to automated update service: Invalid JSON body (500).
  • Direct testing a PR as documented is only possible after merge and GitHub action is available for 5.4-dev branch
  • Installed vim package 😄, forced PHP error and checked it is logged in /var/log/php_errors.log file
  • ⚠️ Compared the 5 new files with [6.0] Add Support for Github Codespaces #45719 already merged 6.0-dev branch there are a lot of differences, I would recommend to unify for 5.4 and 6.0 or at least have only the difference in PHP 8.2 and 8.3

Restrictions and workarounds, as well as the Codespace usage paying needs to be documented. I will do.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45950.

@richard67 richard67 merged commit cecfb91 into joomla:5.4-dev Sep 12, 2025
40 checks passed
@richard67
Copy link
Member

Thanks all (author and testers).

@richard67 richard67 added this to the Joomla! 5.4.0 milestone Sep 12, 2025
@alikon alikon deleted the backport-codespaces branch September 13, 2025 08:18
dgrammatiko pushed a commit to dgrammatiko/joomla-cms that referenced this pull request Jan 25, 2026
Backport [6.0] Add Support for Github Codespaces joomla#45719 to 5.4
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.

4 participants