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

Add step-by-step guide on transferring repos into the organization #68

Merged
merged 6 commits into from
Dec 18, 2018

Conversation

joyeecheung
Copy link
Member

Fixes: #63

The first commit moves the travis integration guide in nodejs/automation here.

The second commit moves the npm management documentation here.

The third commit adds a step-by-step guide on transferring repositories into the organization.

nodejs/TSC#495 (updating the repo management policy from "a simple majority is needed to transfer a repo" to "approval after 72 hours without objection") should land before this could land.

I can open PRs in the automation repo and the build repo to update the original docs with a pointer to the documents here after this lands.

@joyeecheung joyeecheung changed the title Repo transfer Add step-by-step guide on transferring repos into the organization Feb 20, 2018
@joyeecheung
Copy link
Member Author

cc @nodejs/tsc @nodejs/community-committee

joyeecheung

This comment was marked as off-topic.

mcollina

This comment was marked as off-topic.

targos

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

mcollina

This comment was marked as off-topic.

* The purpose of the `nodejs-foundation` user is not to enable Build
Workgroup members to publish npm modules, that should be left to the
module collaborators.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We might also can mention that developers should enable 2FA on npm as well here.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@watilde The foundation account does not even enable 2FA on npm (can we fix that?) ...so probably not really worth mentioning that for now

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I thought it's the same situation with @nodejs-github-bot account without 2FA in GitHub and collaborators need to enable their 2FA. We can mention later after landing this.

@joyeecheung
Copy link
Member Author

This has dragged long enough....sorry. I removed the guide on travis since we can now use the travis GitHub app - I'll add a new guide later (or maybe @targos or @Trott can do that?). Meanwhile, I think we can land the transfer guide and the npm management documentation first.

PTAL.



[`nodejs-foundation`]: https://www.npmjs.com/~nodejs-foundation
[secrets repo]: https://github.com/nodejs/secrets/tree/master/test/test_credentials.md
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a dead link

npm-management.md Outdated Show resolved Hide resolved
npm-management.md Outdated Show resolved Hide resolved
Co-Authored-By: joyeecheung <joyeec9h3@gmail.com>
@joyeecheung
Copy link
Member Author

Addressed some of the reviews from @watilde and @targos.

I think this should be ready to land?

Copy link
Member

@targos targos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM once the link is fixed

npm-management.md Outdated Show resolved Hide resolved
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Co-Authored-By: joyeecheung <joyeec9h3@gmail.com>
@joyeecheung joyeecheung merged commit 0f2af76 into nodejs:master Dec 18, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants