Skip to content

pkgjs/* and nodejs/ci-config* repo housekeeping #348

Closed

Description

@dominykas

I'm working on this - this is to keep track of things.

All repos need to have:

Activity

  1. changed the title [-]pkgjs and ci-config* repo housekeeping[/-] [+]pkgjs/* and nodejs/ci-config* repo housekeeping[/+] on Apr 21, 2020
  2. jonchurch commented on Apr 21, 2020

    @jonchurch
    Contributor

    Perhaps it's more effective to use the default community health file method of creating a .github repo where these standard documents live and are applied to all repos under the org?

    I am not certain that it applies retroactively for existing repos, but that is my understanding.

  3. ljharb commented on Apr 21, 2020

    @ljharb
    SponsorMember

    It will if they don't have the file in the repo already.

  4. dominykas commented on Apr 21, 2020

    @dominykas
    MemberAuthor

    Hmmm. That's obviously a lot more efficient than having them at the root of each repo, but is that sufficient from node.js org repo requirements perspective? Who would be able to get that done at node.js org level so that we don't have to do it in ci-config* repos?

  5. wesleytodd commented on Apr 22, 2020

    @wesleytodd
    Member

    I think it is fine to have a hybrid to start. So having them in some repos (ones which already have them) but then creating .github repo for the rest. In the pkgjs org I think I would rather see us go this route so we don't go through this for every new repo.

  6. mhdawson commented on Apr 24, 2020

    @mhdawson
    Member

    Once we prove out the concept in the pkgjs org I think we should also explore doing it in the Node.js org as well. It think it would greatly increase the likelihood of new repos following the requirements. But for now I'd just do the ci-* ones manually.

  7. bnb commented on Apr 27, 2020

    @bnb
    Contributor

    Huge +1 to using the .github repo approach and potentially exploring it for the nodejs GitHub org once we've seen it working effectively in pkgjs.

  8. dominykas commented on Apr 30, 2020

    @dominykas
    MemberAuthor

    OK, so here's pkgjs/.github#1 - once merged, I'll see if my other PRs need an update (I think we should remove the per-repo files?) or close them.

  9. dominykas commented on May 3, 2020

    @dominykas
    MemberAuthor

    Adding to agenda, primarily to answer two questions:

    1. Need approvals for Add CoC, contribution guidelines and PR template pkgjs/.github#1
    2. Do we want to update existing repos to remove the existing templates?
  10. jonchurch commented on May 15, 2020

    @jonchurch
    Contributor

    One thing to consider that I don't think I mentioned anywhere, I believe that when using the default .github approach, those files are not included when the repo is cloned. I don't know if that matters at all, and I'd still prefer the .github approach, but it is a difference in behavior to note.

  11. dominykas commented on Jul 15, 2020

    @dominykas
    MemberAuthor

    OK, so what I did:

    • CoC is included in .github repo, but also at the root of the repo (and removed from .github folder where it was present). While GH does handily provide a link to CoC when commenting/opening issues/PRs, I feel that the presence of the file at the root is important for additional visibility (and reassurance).
    • I personally think that it's probably enough to have CONTRIBUTING.md in the .github repo, but the governance doc says it has to be in the repo, so I added it in the repo, at the root level, because I suspect it's more useful there, than hidden as a small link somewhere else.
    • I kept the PR template in .github - I think that's good enough, and possibly each repo can add more details to it, but this satisfies the requirement?
  12. dominykas commented on Aug 28, 2020

    @dominykas
    MemberAuthor

    All PRs now merged, items ticked off - closing.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions