User tests: Successful: Unsuccessful:
Pull Request for Issue # .
Handle edge case with lead to null then error when arguments does not match what is expected for example when provided arguments are less the 1 or 2
No more errors for this specific edge case (on argument count mismatch). Basically it returns null as soon as possible.
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
Status | New | ⇒ | Pending |
Category | ⇒ | Repository |
Thanks, can we reproduce this somehow in core? Even when I have to modify some files to see it in action.
I'm fixing cs fixer issue in ci. My local setup had the old coding standards. I think now Joomla core uses PSR-12 right?
Labels |
Added:
PR-5.2-dev
|
Now all ci tests pass
Here is something I tried today to automate everything as much as possible. Mainly focused on tech-savvy people or developers alike:
git clone https://github.com/joomla/joomla-cms.git --depth 1 --branch 5.2-dev j5.2-dev+pr.44650
cd j5.2-dev+pr.44650
composer install && npm ci
php installation/joomla.php install
php cli/joomla.php core:update:channel custom --url=https://artifacts.joomla.org/drone/joomla/joomla-cms/5.2-dev/44650/downloads/80970/pr_list.xml
php cli/joomla.php core:update
Now with this steps one should have the correct pr in their local step or anywhere else.
I'm just sharing . I'm not saying in any way that's what should be done.
@brianteeman This is exactly what I wanted to do for joomla community thanks to awesome Super Joomlers that contribute everyday
Thanks, can we reproduce this somehow in core? Even when I have to modify some files to see it in action.