Bug#513053: [checks/descriptions] warn when first person is used in descriptions
Raphael Geissert <atomo64@gmail.com> writes:
> I originally used severity: normal, but later was unsure about the
> severity because if I filed a bug report I would use severity: minor but
> using this severity causes an EWI code of I which lintian by default
> doesn't display it.
These patch sets have a lot of this sort of multiple commits in them.
Could you use git rebase --interactive before generating the mailbox to
squash some of these commits together when they're simple corrections to
previous diffs rather than new work?
I'll take care of compressing this one down.
> As a bonus the attached mbox contains a patch implementing:
>
>> Tag: first-word-of-sentences-too-repetitive
>> Severity: minor
>> Certainty: wild-guess
>> Info: The given word is used very often as the first word of a sentence.
>> This makes the description hard to read because it is repetitive.
>> .
>> Consider rephrasing the description, using a more wide diversity of words.
>> In case you have troubles with it an email to the english i18n mailing
>> list should help.
Hm, this one fails my "would I find this helpful or just annoying if it
triggered on one of my packages" test. It feels very picky to me, even
too picky for --pedantic, and getting into the area where automated
analysis just isn't helpful compared to a native speaker reviewing the
text. But this is a "soft" opinion -- I'm willing to be outvoted if other
people don't agree with me.
--
Russ Allbery (rra@debian.org) <http://www.eyrie.org/~eagle/>
Reply to: