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
docs: add pull request section to CONTRIBUTING.md #2202
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 💯 🚀 ..
CONTRIBUTING.md
Outdated
|
||
### Submitting Pull Requests | ||
|
||
If you've cloned the project directly and are working on a feature branch, you'll need write permissions to be able to push that branch to the remote to open a pull request. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- This statement is obvious enough.
- I don't see a reason to share write permission with contributors. Fork is only right way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see a reason to share write permission with contributors. Fork is a only right way.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, sorry, we should definitely not be recommending a clone of lerna/lerna
itself. Github's "standard" fork model is the way to go.
Sounds good! Thanks @3imed-jaberi @dizel3d @evocateur. I cam amend the PR to encourage forking. |
@piperchester you may also add the link to respective github guide |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks!
Motivation and Context
This is something I ran into when beginning work on #2198. I'm happy to update the docs to only allow forking if that's the contribution model we'd rather see here, but wanted to start the discussion.