Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP - doc: allow multiple orgs and add pkgjs org #484

Merged
merged 3 commits into from
Oct 14, 2022

Conversation

mhdawson
Copy link
Member

Refs: #470

Signed-off-by: Michael Dawson [email protected]

@mhdawson
Copy link
Member Author

mhdawson commented Mar 26, 2020

This is first part of several that are needed for #470. Should not land until those have been covered off:

  • changes to the org management policy (this PR)
  • Proposed charter for Package Maintenance team which delegates approval of the creation of
    new repos to that team, addition of members, along with some org owner capability. I believe this
    is what the package maintenance team wants, but need to work through the details.
  • check if any docs in the moderation repo need to be updated. - PR for updates - https://github.com/nodejs/moderation/pull/531
  • check that existing repos in pkgjs org have required licences, governance etc.
  • Add current org owners as owners to the pkgjs org, adjust existing owners to reflect that is
    outlined in the charter for the Package Maintenance team.
  • update onboarding docs for TSC (possibly CommComm as well) to add them as owners where
    appropriate.

@mcollina
Copy link
Member

I think you need to update the onboarding docs for TSC and CommComm members as they will become org owners.

@mhdawson
Copy link
Member Author

@mcollina thanks, will add that to the list.

@mmarchini
Copy link
Contributor

mmarchini commented Sep 16, 2020

One thing I noticed (and don't have a good fix for) is that Node.js members won't be able to ping teams on the pkgjs org. Not sure if that's something we didn't consider or if it was intentional, and unfortunately I don't think there are any workarounds for it, but might be a good thing to add to a "considerations list" or something for folks thinking about spinning up the next separate org.

@dominykas
Copy link
Member

This is a good point. Does github have an issue tracker for things like this? Node/pkgjs is not the only place where I'm missing a sort of "umbrella org" / cross-org teams feature. At a certain point, esp. in the ecosystem of loooooots of small modules, a single org becomes too large to administer. I sure we're not the only ones experiencing this.

@mmarchini
Copy link
Contributor

There's a private repo, but it's more for feedback on beta features instead of new features. Sending the suggestion via support might be a good idea.

Base automatically changed from master to main March 18, 2021 15:55
@bnb
Copy link
Contributor

bnb commented Dec 29, 2021

@mhdawson do we want to continue with this?

@mhdawson
Copy link
Member Author

mhdawson commented Jan 4, 2022

@bnb yes, since we already have the second org :)

@mhdawson
Copy link
Member Author

mhdawson commented Jan 4, 2022

@nodejs/tsc any objections to this. If not I'll rebase and submit another PR to update the on/offboarding docs

@bnb would you be able to check over the moderation docs to see if they need any changes?

@mmarchini
Copy link
Contributor

should we move the checklist from #484 (comment) to a new issue to make sure it doesn't get lost?

@bnb
Copy link
Contributor

bnb commented Jan 14, 2022

@mhdawson making "Node.js GitHub Organization" be "Node.js GitHub Organizations" in every instance should cover it.

@mhdawson
Copy link
Member Author

@bnb

making "Node.js GitHub Organization" be "Node.js GitHub Organizations" in every instance should cover it.

Is that something you'd like to do or should I submit a PR to do that?

@bnb
Copy link
Contributor

bnb commented Jan 14, 2022

@mhdawson i'm happy to do it in a separate PR, just wasn't sure if you wanted to do it in this one.

@mhdawson
Copy link
Member Author

@bnb I was thinking the docs might be in the moderation repo, if its in this repo then probably makes sense to add to this PR. Since you just looked was it just this doc ? - https://github.com/nodejs/admin/blob/main/Moderation-Policy.md

@mhdawson
Copy link
Member Author

@bnb added the updates to the Moderation-Policy.md and one small PR to the docs in the moderation repo - https://github.com/nodejs/moderation/pull/531 once that lands I think this PR can land.

@Trott
Copy link
Member

Trott commented Sep 29, 2022

@mhdawson If this is still needed, can you resolve the conflict?

@bnb
Copy link
Contributor

bnb commented Sep 29, 2022

IMO it is still needed. https://github.com/nodejs/moderation/pull/531 has also landed, which should theoretically unblock :)

@mhdawson
Copy link
Member Author

@Trott, I rebased and will now land.

@mhdawson mhdawson merged commit 933dd6c into nodejs:main Oct 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants