mbox series

[v2,0/2] Add bblock documentation

Message ID 20241104-add-bblock-documentation-v2-0-74a8a2cff0f8@baylibre.com
Headers show
Series Add bblock documentation | expand

Message

Julien Stephan Nov. 4, 2024, 3:59 p.m. UTC
Hello,

The main purpose of this series is to add some information for the user
the use VALE and SPHINXLINT on a subset of the documentation.

Signed-off-by: Julien Stephan <jstephan@baylibre.com>
---
Changes in v2:
- Fixed various typos reported by Ulrich Ölmann  and Antonin Godard
- Split series: bblock documetnation was sent into a separate series
- Link to v1: https://lore.kernel.org/r/20241031-add-bblock-documentation-v1-0-32b89093bbda@baylibre.com

---
Julien Stephan (2):
      README: add instruction to run Vale on a subset
      documentation: Makefile: add SPHINXLINTDOCS to specify subset to sphinx-lint

 documentation/Makefile | 23 ++++++++++++-----------
 documentation/README   | 18 +++++++++++++++++-
 2 files changed, 29 insertions(+), 12 deletions(-)
---
base-commit: 9563855ccd92e21fb6f8320c96a3a83e115c947e
change-id: 20241031-add-bblock-documentation-4a5f8cf9c6d2

Best regards,

Comments

Antonin Godard Nov. 5, 2024, 1:12 p.m. UTC | #1
Hi Julien,

On Mon Nov 4, 2024 at 4:59 PM CET, Julien Stephan wrote:
> Hello,
>
> The main purpose of this series is to add some information for the user
> the use VALE and SPHINXLINT on a subset of the documentation.
>
> Signed-off-by: Julien Stephan <jstephan@baylibre.com>
> ---
> Changes in v2:
> - Fixed various typos reported by Ulrich Ölmann  and Antonin Godard
> - Split series: bblock documetnation was sent into a separate series
> - Link to v1: https://lore.kernel.org/r/20241031-add-bblock-documentation-v1-0-32b89093bbda@baylibre.com
>
> ---
> Julien Stephan (2):
>       README: add instruction to run Vale on a subset
>       documentation: Makefile: add SPHINXLINTDOCS to specify subset to sphinx-lint
>
>  documentation/Makefile | 23 ++++++++++++-----------
>  documentation/README   | 18 +++++++++++++++++-
>  2 files changed, 29 insertions(+), 12 deletions(-)
> ---
> base-commit: 9563855ccd92e21fb6f8320c96a3a83e115c947e
> change-id: 20241031-add-bblock-documentation-4a5f8cf9c6d2
>
> Best regards,

Looks good, thanks, this will be useful!

For the series:

Reviewed-by: Antonin Godard <antonin.godard@bootlin.com>
Tested-by: Antonin Godard <antonin.godard@bootlin.com>

Cheers,
Antonin
Quentin Schulz Nov. 5, 2024, 1:14 p.m. UTC | #2
Hi Antonin,

On 11/5/24 2:12 PM, Antonin Godard via lists.yoctoproject.org wrote:
> Hi Julien,
> 
> On Mon Nov 4, 2024 at 4:59 PM CET, Julien Stephan wrote:
>> Hello,
>>
>> The main purpose of this series is to add some information for the user
>> the use VALE and SPHINXLINT on a subset of the documentation.
>>
>> Signed-off-by: Julien Stephan <jstephan@baylibre.com>
>> ---
>> Changes in v2:
>> - Fixed various typos reported by Ulrich Ölmann  and Antonin Godard
>> - Split series: bblock documetnation was sent into a separate series
>> - Link to v1: https://lore.kernel.org/r/20241031-add-bblock-documentation-v1-0-32b89093bbda@baylibre.com
>>
>> ---
>> Julien Stephan (2):
>>        README: add instruction to run Vale on a subset
>>        documentation: Makefile: add SPHINXLINTDOCS to specify subset to sphinx-lint
>>
>>   documentation/Makefile | 23 ++++++++++++-----------
>>   documentation/README   | 18 +++++++++++++++++-
>>   2 files changed, 29 insertions(+), 12 deletions(-)
>> ---
>> base-commit: 9563855ccd92e21fb6f8320c96a3a83e115c947e
>> change-id: 20241031-add-bblock-documentation-4a5f8cf9c6d2
>>
>> Best regards,
> 
> Looks good, thanks, this will be useful!
> 
> For the series:
> 
> Reviewed-by: Antonin Godard <antonin.godard@bootlin.com>
> Tested-by: Antonin Godard <antonin.godard@bootlin.com>
> 

There's a v3 already BTW, so maybe take/review that one instead?

Cheers,
Quentin
Antonin Godard Nov. 5, 2024, 1:27 p.m. UTC | #3
Hi Quentin

On Tue Nov 5, 2024 at 2:14 PM CET, Quentin Schulz wrote:
[...]
>
> There's a v3 already BTW, so maybe take/review that one instead?
>
> Cheers,
> Quentin

The original series was split in two, and only the other half was sent as a v3 I
believe. Thanks for your vigilance! :)

Cheers,
Antonin
Quentin Schulz Nov. 5, 2024, 1:33 p.m. UTC | #4
On 11/5/24 2:27 PM, Antonin Godard wrote:
> Hi Quentin
> 
> On Tue Nov 5, 2024 at 2:14 PM CET, Quentin Schulz wrote:
> [...]
>>
>> There's a v3 already BTW, so maybe take/review that one instead?
>>
>> Cheers,
>> Quentin
> 
> The original series was split in two, and only the other half was sent as a v3 I
> believe. Thanks for your vigilance! :)
> 

Oof, two different series with the same title for the cover letter? I 
wouldn't even have read the v2 at all not knowing it was a different 
series of patches!

Cheers,
Quentin
Julien Stephan Nov. 5, 2024, 1:48 p.m. UTC | #5
Hi Quentin and Antonin,

Oops sorry I splitted the series in two, but I forgot to update the
cover letter title of one of them :/

Cheers
Julien

Le mar. 5 nov. 2024 à 14:33, Quentin Schulz <quentin.schulz@cherry.de> a écrit :
>
> On 11/5/24 2:27 PM, Antonin Godard wrote:
> > Hi Quentin
> >
> > On Tue Nov 5, 2024 at 2:14 PM CET, Quentin Schulz wrote:
> > [...]
> >>
> >> There's a v3 already BTW, so maybe take/review that one instead?
> >>
> >> Cheers,
> >> Quentin
> >
> > The original series was split in two, and only the other half was sent as a v3 I
> > believe. Thanks for your vigilance! :)
> >
>
> Oof, two different series with the same title for the cover letter? I
> wouldn't even have read the v2 at all not knowing it was a different
> series of patches!
>
> Cheers,
> Quentin