Skip to content

prep --check: Add support to spellcheck is codespell is installed#68

Open
marcosps wants to merge 1 commit into
mricon:masterfrom
marcosps:check-codespell
Open

prep --check: Add support to spellcheck is codespell is installed#68
marcosps wants to merge 1 commit into
mricon:masterfrom
marcosps:check-codespell

Conversation

@marcosps

@marcosps marcosps commented Oct 2, 2025

Copy link
Copy Markdown
Contributor

This commit add --codespell to checkpatch.pl is the codespell package is install.

If b4 is installed with pipx you need to use --system-site-packages when calling install, otherwise it won't see the packages installed by your package manager.

@mricon this new feature was added since I forgot to run checkpatch.pl --codespell when sending the patch to the ML. I believed that b4 would do such check, but I was mistaken. What do you think about it?

This commit add --codespell to checkpatch.pl is the codespell package is
install.

If b4 is installed with pipx you need to use --system-site-packages when
calling install, otherwise it won't see the packages installed by your
package manager.

Signed-off-by: Marcos Paulo de Souza <mpdesouza@suse.com>
@marcosps

Copy link
Copy Markdown
Contributor Author

@mricon ping :)

@marc-hb

marc-hb commented Oct 30, 2025

Copy link
Copy Markdown

From: #62 (comment)

This looks good, but can you please send it as a patch series to tools@kernel.org? This way it's transparent for everyone else and not just locked inside github.

Also from: 220ba25

Please send patches to tools@kernel.org.

Using b4 to send b4 patches is not a shocking demand :-D (I was wondering about exactly that)

@marcosps

marcosps commented Nov 7, 2025

Copy link
Copy Markdown
Contributor Author

@marc-hb never thought about it, but really makes sense. Just sent the patch, thanks for letting me know :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants