lcov-reporter-action icon indicating copy to clipboard operation
lcov-reporter-action copied to clipboard

PR Body Too Long

Open LiamMorrow opened this issue 4 years ago • 4 comments

When running on a medium to large project the output is too large to fit in a PR comment.

   request: { hook: [Function: bound bound register], validate: [Object] } },
   errors: [
     {
       resource: 'IssueComment',
       code: 'unprocessable',
       field: 'data',
       message: 'Body is too long (maximum is 65536 characters)'
     }
   ],
   documentation_url: 'https://developer.github.com/v3/issues/comments/#create-a-comment'
 }
 ##[error]Validation Failed: {"resource":"IssueComment","code":"unprocessable","field":"data","message":"Body is too long (maximum is 65536 characters)"}

It would be good to have an option which shows a stripped or truncated version (maybe just top level projects?). I'm not too familiar with lcov format to determine how feasible this is.

LiamMorrow avatar May 05 '20 07:05 LiamMorrow

Have the same issue. I think truncated version would be good.

bartlomiejzuber avatar May 21 '20 05:05 bartlomiejzuber

Same issue here :(

mdrgeniebelt avatar Jun 22 '20 11:06 mdrgeniebelt

My solution was to just modify the code to have the total coverage, but not the full coverage:

exports.comment = function comment(percentage, options) {
  return fragment(
    `Coverage after merging ${b(options.head)} into ${b(options.base)}`,
    table(tbody(tr(th(percentage.toFixed(4), '%'))))
  )
}

Would definitely love either a truncated option or maybe even the table is posted in multiple comments

ekeric13 avatar Sep 16 '20 19:09 ekeric13

Opened https://github.com/romeovs/lcov-reporter-action/pull/31 to perhaps address this by allowing an option to only report on changed files.

andybelltree avatar Nov 15 '21 01:11 andybelltree