Added beginner-friendly contribution steps#21211
Added beginner-friendly contribution steps#21211mitaliiiiiiii wants to merge 1 commit intorapid7:masterfrom
Conversation
|
Hi, I have added beginner-friendly steps to improve clarity for new contributors. Please let me know if any changes are needed. |
smcintyre-r7
left a comment
There was a problem hiding this comment.
What's contained within this PR is a basic workflow to commit a single file using git. Nothing within it is specific to Metasploit. If the instructions we have are insufficient for new contributors, then what we might be able to do is reference a beginners guide that could provide even more detail that what is contained here without also duplicating it.
What would you think about removing this section and instead linking to a 3rd party guide that would provide additional details without the duplication?
|
Thanks for your pull request! We require for all contributed code to come from a from a unique branch in your repository before it can be merged. Please create a new branch in your fork of framework and resubmit this from that branch. If you are using Git on the command line that may look like: This helps protect the process, ensure users are aware of commits on the branch being considered for merge, allows for a location for more commits to be offered without mingling with other contributor changes and allows contributors to make progress while a PR is still being reviewed. Please do resubmit from a unique branch, we greatly value your contribution! 🎉 |
Added a beginner-friendly step-by-step contribution guide to the Using-Git.md documentation.
This helps new contributors understand how to fork, clone, create branches, commit changes, and submit pull requests in a simple way.