You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
pkgjs/* and nodejs/ci-config* repo housekeeping #348
A README.md which includes references to the GOVERNANCE.md in the package-maintenance repository as the authoritative governance which applies to the repository.
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.
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?
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.
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.
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.
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?
Activity
[-]pkgjs and ci-config* repo housekeeping[/-][+]pkgjs/* and nodejs/ci-config* repo housekeeping[/+]jonchurch commentedon Apr 21, 2020
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.
ljharb commentedon Apr 21, 2020
It will if they don't have the file in the repo already.
dominykas commentedon Apr 21, 2020
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?
wesleytodd commentedon Apr 22, 2020
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
.githubrepo for the rest. In thepkgjsorg I think I would rather see us go this route so we don't go through this for every new repo.mhdawson commentedon Apr 24, 2020
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.
bnb commentedon Apr 27, 2020
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.
dominykas commentedon Apr 30, 2020
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.
dominykas commentedon May 3, 2020
Adding to agenda, primarily to answer two questions:
jonchurch commentedon May 15, 2020
One thing to consider that I don't think I mentioned anywhere, I believe that when using the default
.githubapproach, 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.githubapproach, but it is a difference in behavior to note.dominykas commentedon Jul 15, 2020
OK, so what I did:
.githubrepo, but also at the root of the repo (and removed from.githubfolder 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).CONTRIBUTING.mdin the.githubrepo, 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..github- I think that's good enough, and possibly each repo can add more details to it, but this satisfies the requirement?dominykas commentedon Aug 28, 2020
All PRs now merged, items ticked off - closing.