b4 icon indicating copy to clipboard operation
b4 copied to clipboard

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

Open marcosps opened this issue 2 months ago • 3 comments

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?

marcosps avatar Oct 02 '25 13:10 marcosps

@mricon ping :)

marcosps avatar Oct 14 '25 13:10 marcosps

From: https://github.com/mricon/b4/pull/62#issuecomment-2950316168

This looks good, but can you please send it as a patch series to [email protected]? This way it's transparent for everyone else and not just locked inside github.

Also from: https://github.com/mricon/b4/commit/220ba25f86b6f295eab8b4e9e67b624e460f0126

Please send patches to [email protected].

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

marc-hb avatar Oct 30 '25 23:10 marc-hb

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

marcosps avatar Nov 07 '25 13:11 marcosps