Bug#513053: [checks/descriptions] warn when first person is used in descriptions
Russ Allbery <rra@debian.org> writes:
> 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.
Oh, and I dropped "You" from the check as it's not first-person and it's
not already checked by debconf. I think that one's more questionable and
more a matter of style. We may want to impose such a style, but I think
more consensus is needed before doing that. I would tend to say "this
package is not suitable for..." instead of "You shouldn't use this package
if...", but I don't think there's anything basically wrong with the second
construction.
--
Russ Allbery (rra@debian.org) <http://www.eyrie.org/~eagle/>
Reply to: