Bug 11780

Summary: Commit message failures should link to commit patch guidelines
Product: [Yocto Project Subprojects] Patchwork/Patchtest Reporter: Joshua Lock <joshuagloe>
Component: PatchtestAssignee: Leonardo Sandoval Gonzalez <leonardo.sandoval.gonzalez>
Status: RESOLVED FIXED QA Contact:
Severity: enhancement    
Priority: Medium CC: bluelightning, denis
Version: unspecified   
Target Milestone: 2.5 M1   
Hardware: x86   
OS: Multiple   
Whiteboard:
OS type for building Yocto: --- Type of Regression: ---
Verified: Documentation change: No (bug/feature does not impact docs)

Description Joshua Lock 2017-07-11 09:46:12 UTC 
Patchtest should help users to resolve any reported issues. 

test_shortlog_format, and other tests of the commit message, should include a link to the "Commit Patch Message Guidelines" on the OE wiki to help the submitter rectify any errors

https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Comment 1 Leonardo Sandoval Gonzalez 2017-07-11 15:44:49 UTC 
I already created a patch [1] but not yet reflected on emails (it is part of master-next to be merged at master < M2). The approach was to append the URL in the footnote area, meaning that all emails coming from pw on behalf of patchest will have the suggested URL. If you think it is better to include depending on the test case, let me know.

[1] http://git.yoctoproject.org/cgit/cgit.cgi/patchtest/commit/?h=master-next&id=cff83280400e4cd8ac4444319e188f9e1b142af9
Comment 2 Leonardo Sandoval Gonzalez 2017-10-09 16:43:04 UTC 
This is already implemented but cannot be resolved because change is still in master-next, a branch pending to be merged with master.
Comment 3 Leonardo Sandoval Gonzalez 2017-10-20 14:49:33 UTC 
Now in master. patchwork's email footer now has the following strings:

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe