Test-infra: Prow issue: People in OWNERS files could not add LGTM

Created on 11 Jul 2019  路  3Comments  路  Source: kubernetes/test-infra

There is a PR in helm/charts. In my opinion, If my partner jpoon add /lgtm label, the PR would be merged. I'm sure that he was in the OWENERS file under approvers and reviewers. Is there has anything we should do?
https://github.com/helm/charts/pull/15354#issuecomment-510305070

picture Aisuko

Most helpful comment

@Aisuko - It appears that your OWNERS file in the mysqlha directory is misspelled OWENERS (note that there is an extra E in the filename). Try renaming that file.

picture clarketm on 11 Jul 2019
👍2

All 3 comments

@Aisuko - It appears that your OWNERS file in the mysqlha directory is misspelled OWENERS (note that there is an extra E in the filename). Try renaming that file.

clarketm picture clarketm on 11 Jul 2019
👍2

Closing since the rename should fix this issue. :)

nikhita picture nikhita on 12 Jul 2019

@clarketm Thanks for your quick reply I'm waiting for my PR merged which to fix the misspelled. 馃槃

Aisuko picture Aisuko on 12 Jul 2019
👍1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

fen4o picture fen4o  路  4Comments
cblecker picture cblecker  路  4Comments
cjwagner picture cjwagner  路  3Comments
BenTheElder picture BenTheElder  路  4Comments
spzala picture spzala  路  4Comments