? Pending

User tests: Successful: Unsuccessful:

avatar brianteeman
brianteeman
3 Apr 2017

Pull Request for Issue #15072 .

avatar brianteeman brianteeman - open - 3 Apr 2017
avatar brianteeman brianteeman - change - 3 Apr 2017
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 3 Apr 2017
Category Libraries
avatar brianteeman
brianteeman - comment - 3 Apr 2017

Looks like travis didnt like the folder being renamed

avatar wilsonge
wilsonge - comment - 3 Apr 2017

I think you did the wrong folder. I think you wanted to make libraries/src/Joomla/CMS uppercase not the old libraries/cms that contains the non-namespaced code in :)

avatar brianteeman
brianteeman - comment - 3 Apr 2017

grrh - blame Dmitri for that

avatar brianteeman brianteeman - close - 3 Apr 2017
avatar brianteeman brianteeman - merge - 3 Apr 2017
avatar brianteeman brianteeman - change - 3 Apr 2017
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2017-04-03 23:03:34
Closed_By brianteeman
Labels Added: ?
avatar zero-24
zero-24 - comment - 3 Apr 2017

Did you realy want to merge it in this state?

avatar brianteeman
brianteeman - comment - 3 Apr 2017

no - it was a mistake - can you revert it please

avatar zero-24
zero-24 - comment - 3 Apr 2017

I'm on my phone and cant see a revert button. If you can see it just hit them ;)

avatar brianteeman
brianteeman - comment - 3 Apr 2017

Guessing I dont have access to revert/undo as I dont see any options - dont want to mess around any more will have to leave it to @wilsonge or @mbabker to undo my mess up - sorry guys

avatar brianteeman
brianteeman - comment - 3 Apr 2017
avatar brianteeman
brianteeman - comment - 3 Apr 2017

actually looked like the code I merged by accident is correct so I will leave it to you to decide to undo it or not

avatar laoneo
laoneo - comment - 4 Apr 2017

I'v reverted it in #15085. Because all the namespaces in the files should be changed as well when all should be uppercase.

Add a Comment

Login with GitHub to post a comment