* [PATCH] docs: maintainer: Fix capitalization
@ 2025-09-15 19:22 Thorsten Blum
2025-09-15 19:29 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Thorsten Blum @ 2025-09-15 19:22 UTC (permalink / raw)
To: Jonathan Corbet; +Cc: Thorsten Blum, workflows, linux-doc, linux-kernel
The sentence starts at the previous line: s/Indicate/indicate/
Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
---
Documentation/maintainer/maintainer-entry-profile.rst | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/Documentation/maintainer/maintainer-entry-profile.rst b/Documentation/maintainer/maintainer-entry-profile.rst
index cda5d691e967..f411a0d10fad 100644
--- a/Documentation/maintainer/maintainer-entry-profile.rst
+++ b/Documentation/maintainer/maintainer-entry-profile.rst
@@ -68,7 +68,7 @@ wait for the next -rc. At a minimum:
submissions should appear before -rc5.
- Last -rc to merge features: Deadline for merge decisions
- Indicate to contributors the point at which an as yet un-applied patch
+ indicate to contributors the point at which an as yet un-applied patch
set will need to wait for the NEXT+1 merge window. Of course there is no
obligation to ever accept any given patchset, but if the review has not
concluded by this point the expectation is the contributor should wait and
--
2.51.0
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: maintainer: Fix capitalization
2025-09-15 19:22 [PATCH] docs: maintainer: Fix capitalization Thorsten Blum
@ 2025-09-15 19:29 ` Randy Dunlap
2025-09-15 19:35 ` Thorsten Blum
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2025-09-15 19:29 UTC (permalink / raw)
To: Thorsten Blum, Jonathan Corbet; +Cc: workflows, linux-doc, linux-kernel
Hi,
On 9/15/25 12:22 PM, Thorsten Blum wrote:
> The sentence starts at the previous line: s/Indicate/indicate/
Are you sure?
ISTM that the entire line is a "sub-heading".
>
> Signed-off-by: Thorsten Blum <thorsten.blum@linux.dev>
> ---
> Documentation/maintainer/maintainer-entry-profile.rst | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/Documentation/maintainer/maintainer-entry-profile.rst b/Documentation/maintainer/maintainer-entry-profile.rst
> index cda5d691e967..f411a0d10fad 100644
> --- a/Documentation/maintainer/maintainer-entry-profile.rst
> +++ b/Documentation/maintainer/maintainer-entry-profile.rst
> @@ -68,7 +68,7 @@ wait for the next -rc. At a minimum:
> submissions should appear before -rc5.
>
> - Last -rc to merge features: Deadline for merge decisions
> - Indicate to contributors the point at which an as yet un-applied patch
> + indicate to contributors the point at which an as yet un-applied patch
> set will need to wait for the NEXT+1 merge window. Of course there is no
> obligation to ever accept any given patchset, but if the review has not
> concluded by this point the expectation is the contributor should wait and
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: maintainer: Fix capitalization
2025-09-15 19:29 ` Randy Dunlap
@ 2025-09-15 19:35 ` Thorsten Blum
2025-09-15 19:43 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Thorsten Blum @ 2025-09-15 19:35 UTC (permalink / raw)
To: Randy Dunlap; +Cc: Jonathan Corbet, workflows, linux-doc, linux-kernel
Hi Randy,
On 15. Sep 2025, at 21:29, Randy Dunlap wrote:
> On 9/15/25 12:22 PM, Thorsten Blum wrote:
>> The sentence starts at the previous line: s/Indicate/indicate/
>
> Are you sure?
> ISTM that the entire line is a "sub-heading".
Hm maybe, but then the layout/rendering on the web is very confusing:
https://docs.kernel.org/maintainer/maintainer-entry-profile.html#key-cycle-dates
How would that be fixed?
Thanks,
Thorsten
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: maintainer: Fix capitalization
2025-09-15 19:35 ` Thorsten Blum
@ 2025-09-15 19:43 ` Randy Dunlap
2025-09-15 20:06 ` Thorsten Blum
0 siblings, 1 reply; 6+ messages in thread
From: Randy Dunlap @ 2025-09-15 19:43 UTC (permalink / raw)
To: Thorsten Blum; +Cc: Jonathan Corbet, workflows, linux-doc, linux-kernel
On 9/15/25 12:35 PM, Thorsten Blum wrote:
> Hi Randy,
>
> On 15. Sep 2025, at 21:29, Randy Dunlap wrote:
>> On 9/15/25 12:22 PM, Thorsten Blum wrote:
>>> The sentence starts at the previous line: s/Indicate/indicate/
>>
>> Are you sure?
>> ISTM that the entire line is a "sub-heading".
After I sent this I saw that I should have added that it's ambiguous
and needs some formatting help, so I'm not surprised to see this:
> Hm maybe, but then the layout/rendering on the web is very confusing:
>
> https://docs.kernel.org/maintainer/maintainer-entry-profile.html#key-cycle-dates
>
> How would that be fixed?
I'm not sure. At a minimum, add another colon at the end of the line.
Or both sub-headings (this one and the one before it) might need a "::"
at the end of the line and then a blank line inserted there.
I'd have to play around with it and test some, but for now I have to do
something else...
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: maintainer: Fix capitalization
2025-09-15 19:43 ` Randy Dunlap
@ 2025-09-15 20:06 ` Thorsten Blum
2025-09-16 3:37 ` Randy Dunlap
0 siblings, 1 reply; 6+ messages in thread
From: Thorsten Blum @ 2025-09-15 20:06 UTC (permalink / raw)
To: Randy Dunlap
Cc: Jonathan Corbet, workflows, linux-doc, linux-kernel, Dan Williams
On 15. Sep 2025, at 21:43, Randy Dunlap wrote:
> On 9/15/25 12:35 PM, Thorsten Blum wrote:
>>
>> On 15. Sep 2025, at 21:29, Randy Dunlap wrote:
>>> On 9/15/25 12:22 PM, Thorsten Blum wrote:
>>>> The sentence starts at the previous line: s/Indicate/indicate/
>>>
>>> Are you sure?
>>> ISTM that the entire line is a "sub-heading".
>
> After I sent this I saw that I should have added that it's ambiguous
> and needs some formatting help, so I'm not surprised to see this:
>
>> Hm maybe, but then the layout/rendering on the web is very confusing:
>>
>> https://docs.kernel.org/maintainer/maintainer-entry-profile.html#key-cycle-dates
I think the entire line is meant to be a sub-heading. If it were a
single sentence, it should be either "Deadline ... indicates" or
"Deadlines ... indicate", but not "Deadline ... indicate".
Maybe the author of 4699c504e603 (cc Dan Williams) can confirm this?
Thanks,
Thorsten
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] docs: maintainer: Fix capitalization
2025-09-15 20:06 ` Thorsten Blum
@ 2025-09-16 3:37 ` Randy Dunlap
0 siblings, 0 replies; 6+ messages in thread
From: Randy Dunlap @ 2025-09-16 3:37 UTC (permalink / raw)
To: Thorsten Blum
Cc: Jonathan Corbet, workflows, linux-doc, linux-kernel, Dan Williams
On 9/15/25 1:06 PM, Thorsten Blum wrote:
> On 15. Sep 2025, at 21:43, Randy Dunlap wrote:
>> On 9/15/25 12:35 PM, Thorsten Blum wrote:
>>>
>>> On 15. Sep 2025, at 21:29, Randy Dunlap wrote:
>>>> On 9/15/25 12:22 PM, Thorsten Blum wrote:
>>>>> The sentence starts at the previous line: s/Indicate/indicate/
>>>>
>>>> Are you sure?
>>>> ISTM that the entire line is a "sub-heading".
>>
>> After I sent this I saw that I should have added that it's ambiguous
>> and needs some formatting help, so I'm not surprised to see this:
>>
>>> Hm maybe, but then the layout/rendering on the web is very confusing:
>>>
>>> https://docs.kernel.org/maintainer/maintainer-entry-profile.html#key-cycle-dates
>
> I think the entire line is meant to be a sub-heading. If it were a
> single sentence, it should be either "Deadline ... indicates" or
> "Deadlines ... indicate", but not "Deadline ... indicate".
>
> Maybe the author of 4699c504e603 (cc Dan Williams) can confirm this?
I think all that is needed is a blank line after each of these sub-headings.
That makes the html output readable and more understandable.
Would you submit that patch, Thorsten?
thanks.
--
~Randy
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2025-09-16 3:37 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2025-09-15 19:22 [PATCH] docs: maintainer: Fix capitalization Thorsten Blum
2025-09-15 19:29 ` Randy Dunlap
2025-09-15 19:35 ` Thorsten Blum
2025-09-15 19:43 ` Randy Dunlap
2025-09-15 20:06 ` Thorsten Blum
2025-09-16 3:37 ` Randy Dunlap
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox