Skip to contentSkip to conversation

patch series

PATCH In review Ryan Desfosses 2 participants 2 messages Apr 19, 2014 PCI

lore
Share & tools
lore
#0 lore raw

cover patch series

There are eight patches in this series. All changes are trivial with no dependencies or required order.

#1 lore raw
Quoted 3 lines · Ryan Desfosses

There are eight patches in this series. All changes are trivial with no dependencies or required order.
--

My small comments:

a. The Subject should have mentioned PCI and trivial.

b. Please use imperative mood for patch descriptions. I.e.,

use remove instead of removed;
use move instead of moved;
use merge instead of merged;
use add instead of added.

Don't resend the patches based only on my comments (unless the PCI maintainer asks you to do so).

thanks,

Signature · 1 line
~Randy