? Pending

User tests: Successful: Unsuccessful:

avatar fastslack
fastslack
16 Nov 2017

Summary of Changes

Moved all libraries names to use namespace

Testing Instructions

Use as usual, all should work fine.

Expected result

No errors

avatar fastslack fastslack - open - 16 Nov 2017
avatar fastslack fastslack - change - 16 Nov 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 16 Nov 2017
Category Libraries Unit Tests
avatar laoneo
laoneo - comment - 11 Dec 2017

There are two unit tests failing. Can you have a look on them? After that it should be fine to get merged.

avatar fastslack fastslack - change - 25 Dec 2017
Labels Added: ? ?
avatar fastslack
fastslack - comment - 25 Dec 2017

@laoneo I'm stuck here, i dont know why there are a conflict if i'm moving that file libraries/joomla/filesystem/path.php to libraries/src/Filesystem/Path.php

avatar mbabker
mbabker - comment - 25 Dec 2017

Because of d88bc6b#diff-ab05fce3c8ea662bd591767ad671c853

You just need to merge the 3.9-dev branch into your branch and git should be able to work itself out (it sees there were modifications to the base file that aren't present in the moved file basically).

avatar joomla-cms-bot joomla-cms-bot - change - 7 Jan 2018
Category Libraries Unit Tests Administration com_associations com_banners com_cache com_categories com_config com_contact com_content com_fields com_finder com_installer com_joomlaupdate com_languages com_login com_media
avatar fastslack fastslack - change - 7 Jan 2018
Labels Removed: ?
avatar joomla-cms-bot joomla-cms-bot - change - 7 Jan 2018
Category Administration com_associations com_banners com_cache com_categories com_config com_contact com_content com_fields com_finder com_installer com_joomlaupdate com_languages com_login com_media Administration com_admin com_associations com_banners com_cache com_categories com_config com_contact com_content com_fields com_finder com_installer com_joomlaupdate
avatar fastslack
fastslack - comment - 7 Jan 2018

Sorry i broke this, im going to open a new PR

avatar fastslack fastslack - change - 7 Jan 2018
Status Pending Closed
Closed_Date 0000-00-00 00:00:00 2018-01-07 17:53:19
Closed_By fastslack
avatar fastslack fastslack - close - 7 Jan 2018
avatar fastslack
fastslack - comment - 8 Jan 2018

Add a Comment

Login with GitHub to post a comment