On Fri, Apr 17, 2020 at 12:04 PM Xiang Xiao <xiaoxiang781...@gmail.com> wrote: > > On Fri, Apr 17, 2020 at 11:17 AM Nathan Hartman > <hartman.nat...@gmail.com> wrote: > > > > On Thu, Apr 16, 2020 at 10:47 PM Xiang Xiao <xiaoxiang781...@gmail.com> > > wrote: > > > > > On Fri, Apr 17, 2020 at 5:54 AM David Sidrane <david.sidr...@nscdg.com> > > > wrote: > > > > > > > > Thank you for fixing the typos. I am sure I can make them faster than > > > > you > > > > can fix them. :) > > > > > > > > > > Actually, checkpatch.sh support -c option which do the spell check by > > > invoke codespell(the same tool used by Linux): > > > https://github.com/codespell-project/codespell > > > How about we enable -c option in github check action? > > > > > > What will happen if the tool detects false positives? > > > > Report some thing like this: > ./nuttx/tools/checkpatch.sh -c -f apps/graphics/littlevgl/lv_conf.h > graphics/littlevgl/lv_conf.h:147: Complier ==> Compiler
We can add the exception to the ignore list, https://manpages.debian.org/testing/codespell/codespell.1.en.html: -I FILE, --ignore-words FILE File that contains words which will be ignored by codespell. File must contain 1 word per line. Words are case sensitive based on how they are written in the dictionary file > > > Cheers, > > Nathan
