[dpdk-ci] [PATCH v4 7/7] tests: add checkpatch

Wei, FangfangX fangfangx.wei at intel.com
Tue Dec 13 09:29:01 CET 2016


>About the title:
>	[dpdk-test-report] |SUCCESS| pw17860 [PATCH, v2] vmxnet3: fix Rx deadlock Please remove the patchwork id pw17860, and remove the comma in [PATCH, v2].

I have remove the comma. 
For the patchwork id "pw17860", I think it would be better to be remained. Because if it's removed from the subject, receivers can't be directly told which patch has been complicated, they must open the email to check it. 

>The label could be shorter.
>"Per-patch" is not needed as every patchwork checks are "per-patch".
>"check" is also redundant.
>I suggest "Intel compilation" as Test-Label.

The label has been changed to "Intel compilation".

>It would be better to have a blank line between the status description and the full report.

A blank line has been added between the status description and the full report.


Best Regards
Fangfang Wei


More information about the ci mailing list