10625 2016-11-08 16:35:07 +0000 Don't show "Series without cover letter" for series without cover letter 2017-12-06 17:15:47 +0000 1 1 6 Yocto Project Subprojects Patchwork/Patchtest Patchwork unspecified All Multiple RESOLVED FIXED Medium+ enhancement 2.3 M1 1 ross.burton jose.a.lamego bluelightning leonardo.sandoval.gonzalez patrick.ohly --- --- No (bug/feature does not impact docs) oldest_to_newest 68098 0 ross.burton 2016-11-08 16:35:07 +0000 If a series of patches is sent without a cover letter then it appears in the series list as "Series without cover letter", which is useless. Showing something like the first patch summary then "(and X more)" would be a start. 68596 1 jose.a.lamego 2016-11-25 15:27:33 +0000 The change in [1] is under review. Once approved, it will be pushed to production. [1] https://lists.yoctoproject.org/pipermail/yocto/2016-November/033207.html 68599 2 leonardo.sandoval.gonzalez 2016-11-25 16:07:04 +0000 (In reply to comment #1) > The change in [1] is under review. Once approved, it will be pushed to > production. > > [1] https://lists.yoctoproject.org/pipermail/yocto/2016-November/033207.html Jose, can you apply this patch on the staging repo so I can see some examples? Another possible series shortlog is to show first n-targets from the series patches. By target I mean the first part of the shorlog before the colon. 68600 3 jose.a.lamego 2016-11-25 16:53:09 +0000 An example of a series sent without cover letter can be seen in https://patchwork-staging.openembedded.org/series/3850/ This is the rationale for the naming choice: - Using the first 30 characters of first series patch's summary plus ellipsis (...) between double quotation marks clearly denotes that the string is a reference to a patch summary, not the current item's. The size limit deals with long summaries to maintain readability at the series list view. Including all the patch targets in the summary could be problematic for large series, and given the fact that many series patches have a common target, it could end up confusing. 68747 4 jose.a.lamego 2016-11-30 17:22:23 +0000 A v2 patch was submitted in [1] to handle one-patch series without cover letter, where the "1/1" prefix caused that the series was named with the default string. [1] https://lists.yoctoproject.org/pipermail/yocto/2016-November/033286.html 68991 5 jose.a.lamego 2016-12-09 17:14:22 +0000 A v4 patch was submitted in [3] to handle one-patch series sent without cover-letter. This patch is merged in the production repository, waiting for the patchwork to be pointed to the repository's HEAD. 69089 6 jose.a.lamego 2016-12-13 17:54:19 +0000 Fix is now implemented. An example of the new series naming is in: https://patchwork.openembedded.org/series/4369/