? Pending

User tests: Successful: Unsuccessful:

avatar PhilETaylor
PhilETaylor
29 Dec 2021

Requesting to be added to code owners for tests/Unit

avatar PhilETaylor PhilETaylor - open - 29 Dec 2021
avatar PhilETaylor PhilETaylor - change - 29 Dec 2021
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 29 Dec 2021
Category Repository
avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2021

Drone failure unrelated to PR content

avatar wilsonge wilsonge - close - 29 Dec 2021
avatar wilsonge wilsonge - merge - 29 Dec 2021
avatar wilsonge wilsonge - change - 29 Dec 2021
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2021-12-29 23:01:38
Closed_By wilsonge
Labels Added: ?
avatar wilsonge
wilsonge - comment - 29 Dec 2021

Enjoy!

avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2021

Gulp! haha Thanks.

avatar HLeithner
HLeithner - comment - 29 Dec 2021

What's the reason for your request?

avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2021

"Code owners are automatically requested for review when someone opens a pull request that modifies code that they own"

Own is a harsh word. Contribute to and have experience of is a better phrase.

Seriously though - something has to be done about the low quality of code in Joomla - if Joomla is to survive as a Modern PHP Application (rather than a shiny frontend) - and Unit Testing is something I have considerable experience in for huge projects. I guess I just had a moment of pity.

There are only a few people providing any unit tests at all recently - and certainly there is a lot to do at the Joomla 4+ level (Im not touching 3 unit tests, I know you are refactoring those)

What is your reason for questioning my involvement?

However:

"The people you choose as code owners must have write permissions for the repository"

Oh well that means this should be reverted ... oh well nice while it lasted..

avatar brianteeman
brianteeman - comment - 29 Dec 2021

it doesnt work if you dont have the correct permissions

avatar PhilETaylor
PhilETaylor - comment - 29 Dec 2021

Like I said.

"The people you choose as code owners must have write permissions for the repository"
Oh well that means this should be reverted ... oh well nice while it lasted..

And I don't want write permissions to this repo. Ever.

avatar HLeithner
HLeithner - comment - 29 Dec 2021

@PhilETaylor that's the reason I asked, with out the permissions it doesn't work. But george should know that...

Anyway yes you are right we need more tests, but we shouldn't do this uncoordinated. Doesn't mean your help isn't appreciated, actually it is.

I revert the pr since as you already found out too it doesn't work as expected.

Add a Comment

Login with GitHub to post a comment