Test-infra: CONTRIBUTING.md link to maintainers cannot be opened by non-members

Created on 20 Jul 2017  路  13Comments  路  Source: kubernetes/test-infra

The contributing docs currently link to kubernetes/test-infra-maintainers which is a dead link.

I'm not actually seeing a suitable replacement yet though.

/cc @ixdy

kindocumentation

Most helpful comment

I wonder why we make such teams private. Any reasons?

GitHub makes them private with no way to make them public, I think.

All 13 comments

From another issue, it seems that the link is not bad, just not visible to non-members.

screen shot 2017-07-20 at 10 29 48 am

Not sure if this should be fixed or is as-intended.

Oh right, I totally forgot that teams aren't visible to non-org-members. We should link to something more useful for new contributors.

@spxtr maybe we should add OWNERS files to test-infra so the munger can assign reviewers automatically?

We should add a CODEOWNERS file so that GitHub assigns reviewers automatically, no need for a munger :)

I think we should strive to have the same developer experience for all of our repos. Having CODEOWNERS in some places and k8s OWNERS in other places is going to be very confusing for contributors.

The fact that test-infra doesn't have a SQ like other repos has already been problematic.

Agree with @ixdy

I keep hearing mention of eventually wanting to drive teams from files vs just leaning on githubs teams but I'm not sure if that's a roadmapped plan

I'm conflicted about this particular case. I do agree that it's nice to have the same developer experience for all the repos, but I don't think we should wall ourselves off from using native GitHub tooling because we made our own a few months before they made theirs.

It's going off on a tangent from this PR though.

I keep hearing mention of eventually wanting to drive teams from files vs just leaning on githubs teams but I'm not sure if that's a roadmapped plan

We'll still be using GitHub teams, we'll just keep the source of truth in files and sync them regularly.

I wonder why we make such teams private. Any reasons?

I wonder why we make such teams private. Any reasons?

GitHub makes them private with no way to make them public, I think.

Not sure what we want to do about this or close, at least we have CODEOWNERS/OWNERS now

/assign

/cc @amwat

/assign @amwat

Was this page helpful?
0 / 5 - 0 ratings

Related issues

cjwagner picture cjwagner  路  3Comments

stevekuznetsov picture stevekuznetsov  路  4Comments

BenTheElder picture BenTheElder  路  4Comments

fejta picture fejta  路  4Comments

cjwagner picture cjwagner  路  3Comments