? ? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
11 Oct 2020

Summary of Changes

Update PHPDoc comment to match method signatures in installation app

Testing Instructions

Code review

avatar PhilETaylor PhilETaylor - open - 11 Oct 2020
avatar PhilETaylor PhilETaylor - change - 11 Oct 2020
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 11 Oct 2020
Category Installation
avatar PhilETaylor PhilETaylor - change - 11 Oct 2020
Labels Added: ?
avatar HLeithner
HLeithner - comment - 11 Oct 2020

cs issues

avatar PhilETaylor
PhilETaylor - comment - 11 Oct 2020

Fixed

avatar ChristineWk ChristineWk - test_item - 12 Oct 2020 - Tested successfully
avatar ChristineWk
ChristineWk - comment - 12 Oct 2020

I have tested this item successfully on 1d6a9bd


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

avatar ceford ceford - test_item - 14 Oct 2020 - Tested successfully
avatar ceford
ceford - comment - 14 Oct 2020

I have tested this item successfully on 1d6a9bd

I really did look at every code change. How sad is that!


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

avatar alikon alikon - change - 14 Oct 2020
The description was changed
Status Pending Ready to Commit
avatar alikon
alikon - comment - 14 Oct 2020

RTC


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

avatar joomla-cms-bot joomla-cms-bot - edited - 14 Oct 2020
avatar PhilETaylor PhilETaylor - change - 15 Oct 2020
Labels Added: ?
avatar PhilETaylor
PhilETaylor - comment - 15 Oct 2020

applied suggestions blindly.

avatar richard67 richard67 - alter_testresult - 15 Oct 2020 - ChristineWk: Tested successfully
avatar richard67 richard67 - alter_testresult - 15 Oct 2020 - ceford: Tested successfully
avatar richard67
richard67 - comment - 15 Oct 2020

Changes after the tests were only code style, indentation, so the previous test results and RTC are still ok. I've added the test result back using the issue tracker.

avatar richard67
richard67 - comment - 15 Oct 2020

I've just finished my pre-merge review. All fine. As soon as drone has passed I'll merge it.

avatar richard67 richard67 - close - 15 Oct 2020
avatar richard67 richard67 - merge - 15 Oct 2020
avatar richard67 richard67 - change - 15 Oct 2020
Status Ready to Commit Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2020-10-15 16:59:59
Closed_By richard67
avatar richard67
richard67 - comment - 15 Oct 2020

Thanks!

Add a Comment

Login with GitHub to post a comment