threads / patch / 53577

patchdocs: mention MyFirstContribution in more places

Subject: [PATCH] docs: mention MyFirstContribution in more places

## tl;dr

7 messages between May 28, 2020 and Jun 8, 2020. Diffs are folded; open one to read it.

replies: 6people: 4as markdown or json

Emily Shaffer· May 28, 2020, 23:49 UTC · lore

While the MyFirstContribution guide exists and has received some use and positive reviews, it is still not as discoverable as it could be. Add a reference to it from the GitHub pull request template, where many brand-new contributors may look. Also add a reference to it in SubmittingPatches, which is the central source of guidance for patch contribution.

Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
---
Philippe mentioned out-of-band that they wished for some more pointers
to MyFirstContribution in the Git documentation
(https://github.com/git/git-scm.com/issues/1464#issuecomment-631757274).

Here's a first pass. The change for the GH PR template was easy, since there are a couple other "here's how" resources listed the same way. But the change for SubmittingPatches I wasn't so sure about. Since that document dives right into technical details and doesn't feature some "other resources" section (at least not that I found), I made a quick reference at the top, which I was hoping would be unintrusive but also easy to find for newbies overwhelmed by the large list of rules. It's not very scalable if we want to add more info, though, so I'm open to other suggestions.

 - Emily
 .github/CONTRIBUTING.md         | 3 +++
 Documentation/SubmittingPatches | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)
Show changes to 2 files +6 −2

.github/CONTRIBUTING.md, Documentation/SubmittingPatches

diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
index e7b4e2f3c2..c8755e38de 100644
--- a/.github/CONTRIBUTING.md
+++ b/.github/CONTRIBUTING.md
@@ -16,4 +16,7 @@ If you prefer video, then [this talk](https://www.youtube.com/watch?v=Q7i_qQW__q
 might be useful to you as the presenter walks you through the contribution
 process by example.
 
+Or, you can follow the ["My First Contribution"](https://git-scm.com/docs/MyFirstContribution)
+tutorial for another example of the contribution process.
+
 Your friendly Git community!
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index 4515cab519..d8c6b4d71b 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -3,8 +3,9 @@ Submitting Patches
 
 == Guidelines
 
-Here are some guidelines for people who want to contribute their code
-to this software.
+Here are some guidelines for people who want to contribute their code to this
+software. There is also an link:MyFirstContribution.html[interactive tutorial]
+available which covers many of these same guidelines.
 
 [[base-branch]]
 === Decide what to base your work on.
-- 
2.27.0.rc0.183.gde8f92d652-goog
Jonathan Nieder· May 29, 2020, 01:06 UTC · re: Emily Shaffer · lore

Re: [PATCH] docs: mention MyFirstContribution in more places

Emily Shaffer wrote:
Show 12 quoted lines
> While the MyFirstContribution guide exists and has received some use and
> positive reviews, it is still not as discoverable as it could be. Add a
> reference to it from the GitHub pull request template, where many
> brand-new contributors may look. Also add a reference to it in
> SubmittingPatches, which is the central source of guidance for patch
> contribution.
>
> Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
> ---
>  .github/CONTRIBUTING.md         | 3 +++
>  Documentation/SubmittingPatches | 5 +++--
>  2 files changed, 6 insertions(+), 2 deletions(-)
Yay!
Show 10 quoted lines
> --- a/.github/CONTRIBUTING.md
> +++ b/.github/CONTRIBUTING.md
> @@ -16,4 +16,7 @@ If you prefer video, then [this talk](https://www.youtube.com/watch?v=Q7i_qQW__q
>  might be useful to you as the presenter walks you through the contribution
>  process by example.
>  
> +Or, you can follow the ["My First Contribution"](https://git-scm.com/docs/MyFirstContribution)
> +tutorial for another example of the contribution process.
> +
>  Your friendly Git community!
Looks good.
Show 11 quoted lines
> --- a/Documentation/SubmittingPatches
> +++ b/Documentation/SubmittingPatches
> @@ -3,8 +3,9 @@ Submitting Patches
>  
>  == Guidelines
>  
> -Here are some guidelines for people who want to contribute their code
> -to this software.
> +Here are some guidelines for people who want to contribute their code to this
> +software. There is also an link:MyFirstContribution.html[interactive tutorial]
> +available which covers many of these same guidelines.

nit: when I see "an interactive tutorial" I imagine a "git tutor" command that interacts with me to guide me through my first contribution (like "vimtutor"). I think this means to just say "a tutorial".

With that tweak,
Reviewed-by: Jonathan Nieder <jrnieder@gmail.com>
Thanks.
Philippe Blain· May 29, 2020, 01:36 UTC · re: Jonathan Nieder · lore

Re: [PATCH] docs: mention MyFirstContribution in more places

Hi Emily, 
Show 48 quoted lines
> Le 28 mai 2020 à 21:06, Jonathan Nieder <jrnieder@gmail.com> a écrit :
> 
> Emily Shaffer wrote:
> 
>> While the MyFirstContribution guide exists and has received some use and
>> positive reviews, it is still not as discoverable as it could be. Add a
>> reference to it from the GitHub pull request template, where many
>> brand-new contributors may look. Also add a reference to it in
>> SubmittingPatches, which is the central source of guidance for patch
>> contribution.
>> 
>> Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
>> ---
>> .github/CONTRIBUTING.md         | 3 +++
>> Documentation/SubmittingPatches | 5 +++--
>> 2 files changed, 6 insertions(+), 2 deletions(-)
> 
> Yay!
> 
>> --- a/.github/CONTRIBUTING.md
>> +++ b/.github/CONTRIBUTING.md
>> @@ -16,4 +16,7 @@ If you prefer video, then [this talk](https://www.youtube.com/watch?v=Q7i_qQW__q
>> might be useful to you as the presenter walks you through the contribution
>> process by example.
>> 
>> +Or, you can follow the ["My First Contribution"](https://git-scm.com/docs/MyFirstContribution)
>> +tutorial for another example of the contribution process.
>> +
>> Your friendly Git community!
> 
> Looks good.
> 
>> --- a/Documentation/SubmittingPatches
>> +++ b/Documentation/SubmittingPatches
>> @@ -3,8 +3,9 @@ Submitting Patches
>> 
>> == Guidelines
>> 
>> -Here are some guidelines for people who want to contribute their code
>> -to this software.
>> +Here are some guidelines for people who want to contribute their code to this
>> +software. There is also an link:MyFirstContribution.html[interactive tutorial]
>> +available which covers many of these same guidelines.
> 
> nit: when I see "an interactive tutorial" I imagine a "git tutor"
> command that interacts with me to guide me through my first
> contribution (like "vimtutor").  I think this means to just say
> "a tutorial".
I agree. Maybe "a step-by-step tutorial" ?
Thanks for taking the time to do this. That's exactly what I had in mind.
Philippe.
Emily Shaffer· Jun 8, 2020, 20:47 UTC · re: Philippe Blain · lore

Re: [PATCH] docs: mention MyFirstContribution in more places

On Thu, May 28, 2020 at 09:36:58PM -0400, Philippe Blain wrote:
Show 53 quoted lines
> 
> Hi Emily, 
> 
> > Le 28 mai 2020 à 21:06, Jonathan Nieder <jrnieder@gmail.com> a écrit :
> > 
> > Emily Shaffer wrote:
> > 
> >> While the MyFirstContribution guide exists and has received some use and
> >> positive reviews, it is still not as discoverable as it could be. Add a
> >> reference to it from the GitHub pull request template, where many
> >> brand-new contributors may look. Also add a reference to it in
> >> SubmittingPatches, which is the central source of guidance for patch
> >> contribution.
> >> 
> >> Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
> >> ---
> >> .github/CONTRIBUTING.md         | 3 +++
> >> Documentation/SubmittingPatches | 5 +++--
> >> 2 files changed, 6 insertions(+), 2 deletions(-)
> > 
> > Yay!
> > 
> >> --- a/.github/CONTRIBUTING.md
> >> +++ b/.github/CONTRIBUTING.md
> >> @@ -16,4 +16,7 @@ If you prefer video, then [this talk](https://www.youtube.com/watch?v=Q7i_qQW__q
> >> might be useful to you as the presenter walks you through the contribution
> >> process by example.
> >> 
> >> +Or, you can follow the ["My First Contribution"](https://git-scm.com/docs/MyFirstContribution)
> >> +tutorial for another example of the contribution process.
> >> +
> >> Your friendly Git community!
> > 
> > Looks good.
> > 
> >> --- a/Documentation/SubmittingPatches
> >> +++ b/Documentation/SubmittingPatches
> >> @@ -3,8 +3,9 @@ Submitting Patches
> >> 
> >> == Guidelines
> >> 
> >> -Here are some guidelines for people who want to contribute their code
> >> -to this software.
> >> +Here are some guidelines for people who want to contribute their code to this
> >> +software. There is also an link:MyFirstContribution.html[interactive tutorial]
> >> +available which covers many of these same guidelines.
> > 
> > nit: when I see "an interactive tutorial" I imagine a "git tutor"
> > command that interacts with me to guide me through my first
> > contribution (like "vimtutor").  I think this means to just say
> > "a tutorial".
> 
> I agree. Maybe "a step-by-step tutorial" ?
Thanks for the suggestion - I like that. I'll send an update shortly.
 - Emily
Emily Shaffer· Jun 8, 2020, 21:11 UTC · re: Emily Shaffer · lore

[PATCH v2] docs: mention MyFirstContribution in more places

While the MyFirstContribution guide exists and has received some use and positive reviews, it is still not as discoverable as it could be. Add a reference to it from the GitHub pull request template, where many brand-new contributors may look. Also add a reference to it in SubmittingPatches, which is the central source of guidance for patch contribution.

Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
---

Since v1, only the tiny nit - took Philippe's advice to refer to a "step-by-step" tutorial, instead of an "interactive" tutorial.

Thanks.
 - Emily
 .github/CONTRIBUTING.md         | 3 +++
 Documentation/SubmittingPatches | 5 +++--
 2 files changed, 6 insertions(+), 2 deletions(-)
Show changes to 2 files +6 −2

.github/CONTRIBUTING.md, Documentation/SubmittingPatches

diff --git a/.github/CONTRIBUTING.md b/.github/CONTRIBUTING.md
index e7b4e2f3c2..c8755e38de 100644
--- a/.github/CONTRIBUTING.md
+++ b/.github/CONTRIBUTING.md
@@ -16,4 +16,7 @@ If you prefer video, then [this talk](https://www.youtube.com/watch?v=Q7i_qQW__q
 might be useful to you as the presenter walks you through the contribution
 process by example.
 
+Or, you can follow the ["My First Contribution"](https://git-scm.com/docs/MyFirstContribution)
+tutorial for another example of the contribution process.
+
 Your friendly Git community!
diff --git a/Documentation/SubmittingPatches b/Documentation/SubmittingPatches
index 4515cab519..ecf9438cf0 100644
--- a/Documentation/SubmittingPatches
+++ b/Documentation/SubmittingPatches
@@ -3,8 +3,9 @@ Submitting Patches
 
 == Guidelines
 
-Here are some guidelines for people who want to contribute their code
-to this software.
+Here are some guidelines for people who want to contribute their code to this
+software. There is also a link:MyFirstContribution.html[step-by-step tutorial]
+available which covers many of these same guidelines.
 
 [[base-branch]]
 === Decide what to base your work on.
-- 
2.27.0.278.ge193c7cf3a9-goog
Philippe Blain· Jun 8, 2020, 21:25 UTC · re: Emily Shaffer · lore

Re: [PATCH v2] docs: mention MyFirstContribution in more places

Hi Emily,
Show 17 quoted lines
> Le 8 juin 2020 à 17:11, Emily Shaffer <emilyshaffer@google.com> a écrit :
> 
> While the MyFirstContribution guide exists and has received some use and
> positive reviews, it is still not as discoverable as it could be. Add a
> reference to it from the GitHub pull request template, where many
> brand-new contributors may look. Also add a reference to it in
> SubmittingPatches, which is the central source of guidance for patch
> contribution.
> 
> Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
> ---
> 
> Since v1, only the tiny nit - took Philippe's advice to refer to a
> "step-by-step" tutorial, instead of an "interactive" tutorial.
> 
> Thanks.
> - Emily
Thanks. Looks good. 
Reviewed-by: Philippe Blain <levraiphilippeblain@gmail.com>
Junio C Hamano· Jun 8, 2020, 22:12 UTC · re: Philippe Blain · lore

Re: [PATCH v2] docs: mention MyFirstContribution in more places

Philippe Blain <levraiphilippeblain@gmail.com> writes:
Show 23 quoted lines
> Hi Emily,
>
>> Le 8 juin 2020 à 17:11, Emily Shaffer <emilyshaffer@google.com> a écrit :
>> 
>> While the MyFirstContribution guide exists and has received some use and
>> positive reviews, it is still not as discoverable as it could be. Add a
>> reference to it from the GitHub pull request template, where many
>> brand-new contributors may look. Also add a reference to it in
>> SubmittingPatches, which is the central source of guidance for patch
>> contribution.
>> 
>> Signed-off-by: Emily Shaffer <emilyshaffer@google.com>
>> ---
>> 
>> Since v1, only the tiny nit - took Philippe's advice to refer to a
>> "step-by-step" tutorial, instead of an "interactive" tutorial.
>> 
>> Thanks.
>> - Emily
>
> Thanks. Looks good. 
>
> Reviewed-by: Philippe Blain <levraiphilippeblain@gmail.com>
Thanks, both.

← back to recent threads