? Pending

User tests: Successful: Unsuccessful:

avatar wojsmol
wojsmol
18 Aug 2018

Pull Request for Issue # .

Summary of Changes

Update .CODEOWNERS after github nick change

Testing Instructions

code review

Expected result

Actual result

Documentation Changes Required

avatar wojsmol wojsmol - open - 18 Aug 2018
avatar wojsmol wojsmol - change - 18 Aug 2018
Status New Pending
avatar joomla-cms-bot joomla-cms-bot - change - 18 Aug 2018
Category Repository
avatar wilsonge wilsonge - change - 18 Aug 2018
Status Pending Fixed in Code Base
Closed_Date 0000-00-00 00:00:00 2018-08-18 14:21:24
Closed_By wilsonge
Labels Added: ?
avatar wilsonge wilsonge - close - 18 Aug 2018
avatar wilsonge wilsonge - merge - 18 Aug 2018
avatar wilsonge
wilsonge - comment - 18 Aug 2018

Always someone who has to change theirs ? thanks!

avatar zero-24
zero-24 - comment - 19 Aug 2018

As he still does not have merge access so it does not matter anyway :)

avatar wojsmol
wojsmol - comment - 19 Aug 2018

@zero-24 This at least ensure that notifications will be send to right person. Without write permissions he can also review, but this review will not be counted by github.

avatar zero-24
zero-24 - comment - 19 Aug 2018

My point is the following:

People with admin or owner permissions can set up a CODEOWNERS file in a repository. The people you choose as code owners must have write permissions for the repository.

https://help.github.com/articles/about-codeowners/ As far as i understand that doc no mail and nor request is send out anyway ;)

avatar brianteeman
brianteeman - comment - 19 Aug 2018

Correct

avatar wojsmol
wojsmol - comment - 19 Aug 2018

One way or the other PR is valid because of nick change, permission part is repo admin thing.

avatar zero-24
zero-24 - comment - 19 Aug 2018

Yes.

Add a Comment

Login with GitHub to post a comment