Re: [Outreachy] Wiki updated needed - Patchset Subject

[Deepak R Varma <drv@mailo.com>]

On Wed, Oct 26, 2022 at 04:00:58AM +0530, Deepak Varma wrote:
> On Mon, Oct 24, 2022 at 02:12:46PM -0700, Alison Schofield wrote:
> > On Sat, Oct 22, 2022 at 10:59:38AM +0530, Deepak R Varma wrote:
> > > On Sun, Oct 23, 2022 at 06:51:11PM -0700, Alison Schofield wrote:
> > > > Greg KH has mentioned this a couple of times lately -
> > > >
> > > > >> Any reason you do not have "staging: vt6655:" as the prefix for
> > > > >> this 0/X email subject line?
> > > > >>
> > > > >> thanks,
> > > > >>
> > > > >> greg k-h
> > > >
> > > > A quick peek in the tutorial tells me it could be made clearer.
> > > > It needs to explicitly say the prefix is required, and then
> > > > make sure any examples do it.
> > >
> > > Hello Alison,
> > > There already is a section named "Patch Subject formatting" on this link [1]
> > > with enough description to determine the patch subject prefix. Can you please
> > > review and confirm where we should improve further to make it clearer??
> > >
> > > 	[1] https://kernelnewbies.org/PatchPhilosophy
> > >
> > > Thank you,
> > > ./drv
> >
> >
> > In https://kernelnewbies.org/Outreachyfirstpatch
> >
> > The example cover letter in 'Submitting a Patchset' has the
> > subject "[RFC 0/7] Update REPORTING-BUGS", with no subsys.
> > It is also confusing because the cover letter is an RFC and
> > the patches are a mix of RFC and PATCH. Very atypical.
> >
> > Let's replace this example with one that was actually submitted
> > and accepted by an Outreachy Intern.  No slight intended to the
> > existing set as it contains an interesting set of changes. However,
> > an example for newbies might better come from newbies.
> >
> > How about a patchset off this list?
> >
> > Would you mind using this one of yours:
> > [PATCH v4 00/11] staging: r8188eu: trivial code cleanup patches
> > but trim it to 3-4 patches plus the cover.
> >
> > It's not a trivial edit, because you need to replace all the
> > example steps with the new patchset content.
>
> Hello Alison,
> I made the changes as suggested and saved those. Looks like they might have
> already gone to production. Sorry about that. I was thinking they will be
> submitted to a moderator for review before releasing,
>
> Could you please review and let me know if any edits are required.

Hello Alison,
Did you get a chance to review the edits I made? Please share your feedback so
that I can proceed with the next changes to the tutorial page.

Thank you,
./drv

>
> Thank you,
> ./drv
>
> >
> > Thanks,
> > Alison
> >
> > >
> > > >
> > > > Thanks,
> > > > Alison
> > > >
> > >
> > >