Re: [libvirt PATCH 0/2] Fix some typos and spelling mistakes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On a Wednesday in 2021, Tim Wiederhake wrote:
Found by codespell [1].

It would be nice to automate this, but I could not get codespell to
also check code comments,

You can automate the spellchecking, but sometimes typos are included in
sentences that should be rewritten completely - you cannot (yet?)
automatically check for that.

Also, checking comments outside of public APIs is not that important - the
code will usually be harder to read than the comment with a typo.

and ran into a known issue [2] with codespell
that prevents putting some terms with mixed letter cases on the skip list.


With false positives and no way to silence the warnings, I'm afraid the
automation would not be useful.

Jano

[1] https://github.com/codespell-project/codespell
[2] https://github.com/codespell-project/codespell/issues/1860

Tim Wiederhake (2):
 Fix spelling
 virlog: Remove stray "todo" in comment

Attachment: signature.asc
Description: PGP signature


[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]

  Powered by Linux