Skip to contentSkip to conversation

README: add mentions on shellcheck and reuse

PATCH Applied Bartosz Golaszewski 2 participants 4 messages Jun 13, 2024 – Jun 14, 2024 GPIO

lore
Share & tools
lore
#0 lore raw

PATCH 1/1 README: add mentions on shellcheck and reuse

From: Bartosz Golaszewski <[email protected]>

Extend the "Contributing" section of the README to include mentions of shell scripts having to pass the shellcheck test and the entire tree having to conform to reuse lint requirements.

---
 README | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)
README +13 -9
index 7a92a49..a6f24d1 100644
@@ -284,7 +284,9 @@ to send plain text[3].
 
 Code submissions should stick to the linux kernel coding style[4] and
 follow the kernel patch submission process[5] as applied to the libgpiod
-source tree.
+source tree. All shell scripts must pass `shellcheck` tests[9]. All files
+must have a license and copyright SPDX headers and the repo is expected to
+pass the `reuse lint` check[10].
 
 The mailing list archive[6] contains all the historical mails to the list,
 and is the place to check to ensure your e-mail has been received.
@@ -300,11 +302,13 @@ tarballs should be fetched from kernel.org[8].
 
 For more information, refer to CONTRIBUTING.md in this repository.
 
-[1] https://github.com/kward/shunit2
-[2] http://vger.kernel.org/vger-lists.html#linux-gpio
-[3] https://docs.kernel.org/process/email-clients.html
-[4] https://docs.kernel.org/process/coding-style.html
-[5] https://docs.kernel.org/process/submitting-patches.html
-[6] https://lore.kernel.org/linux-gpio/
-[7] https://github.com/brgl/libgpiod
-[8] https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
+[1]  https://github.com/kward/shunit2
+[2]  http://vger.kernel.org/vger-lists.html#linux-gpio
+[3]  https://docs.kernel.org/process/email-clients.html
+[4]  https://docs.kernel.org/process/coding-style.html
+[5]  https://docs.kernel.org/process/submitting-patches.html
+[6]  https://lore.kernel.org/linux-gpio/
+[7]  https://github.com/brgl/libgpiod
+[8]  https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
+[9]  https://www.shellcheck.net/
+[10] https://reuse.software/
Signature · 1 line
2.43.0
#1 lore raw

Bartosz,

Quoted 4 lines · Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Extend the "Contributing" section of the README to include mentions of

I think the content is good and will help clear up some confusion for future contributors. However, it's a little confusing to have contribution guidelines in the README and then have it point to CONTRIBUTING.md which duplicates some information but doesn't actually have much more information.

If I had a vote, it'd be to consolidate the information into one spot, but maybe that should be done in a subsequent patch?

Quoted 48 lines

shell scripts having to pass the shellcheck test and the entire tree having to conform to reuse lint requirements.

---
 README | 22 +++++++++++++---------
 1 file changed, 13 insertions(+), 9 deletions(-)
README +13 -10
index 7a92a49..a6f24d1 100644
@@ -284,7 +284,9 @@ to send plain text[3].
 
 Code submissions should stick to the linux kernel coding style[4] and
 follow the kernel patch submission process[5] as applied to the libgpiod
-source tree.
+source tree. All shell scripts must pass `shellcheck` tests[9]. All files
+must have a license and copyright SPDX headers and the repo is expected to
+pass the `reuse lint` check[10].
 
 The mailing list archive[6] contains all the historical mails to the list,
 and is the place to check to ensure your e-mail has been received.
@@ -300,11 +302,13 @@ tarballs should be fetched from kernel.org[8].
 
 For more information, refer to CONTRIBUTING.md in this repository.
 
-[1] https://github.com/kward/shunit2
-[2] http://vger.kernel.org/vger-lists.html#linux-gpio
-[3] https://docs.kernel.org/process/email-clients.html
-[4] https://docs.kernel.org/process/coding-style.html
-[5] https://docs.kernel.org/process/submitting-patches.html
-[6] https://lore.kernel.org/linux-gpio/
-[7] https://github.com/brgl/libgpiod
-[8] https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
+[1]  https://github.com/kward/shunit2
+[2]  http://vger.kernel.org/vger-lists.html#linux-gpio
+[3]  https://docs.kernel.org/process/email-clients.html
+[4]  https://docs.kernel.org/process/coding-style.html
+[5]  https://docs.kernel.org/process/submitting-patches.html
+[6]  https://lore.kernel.org/linux-gpio/
+[7]  https://github.com/brgl/libgpiod
+[8]  https://mirrors.edge.kernel.org/pub/software/libs/libgpiod/
+[9]  https://www.shellcheck.net/
+[10] https://reuse.software/
--

2.43.0

#2 lore raw
Quoted 9 lines · Vincent Fazio

Bartosz,

Quoted 4 lines · Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Extend the "Contributing" section of the README to include mentions of

Quoted 8 lines

I think the content is good and will help clear up some confusion for future contributors. However, it's a little confusing to have contribution guidelines in the README and then have it point to CONTRIBUTING.md which duplicates some information but doesn't actually have much more information.

If I had a vote, it'd be to consolidate the information into one spot, but maybe that should be done in a subsequent patch?

Ok, I can send a follow up that moves most of this section to the separate file.

Bart

author Applied
#3 lore raw
In reply to Bartosz Golaszewski

From: Bartosz Golaszewski <[email protected]>

Quoted 5 lines · Bartosz Golaszewski

Extend the "Contributing" section of the README to include mentions of shell scripts having to pass the shellcheck test and the entire tree having to conform to reuse lint requirements.

Applied, thanks!

[1/1] README: add mentions on shellcheck and reuse

      commit: d7f2bb23b532dfcac7f2178abe148f5aed91248b

Best regards,

Signature · 1 line
Bartosz Golaszewski <[email protected]>