@@ -485,43 +485,63 @@ def generate_html_report(
485485 )
486486
487487 if r .issues :
488- parts .append (
489- '<tr><td style="padding:0;">'
490- '<table role="presentation" width="100%" cellpadding="0"'
491- ' cellspacing="0" style="font-size:12px;">'
492- '<tr style="background-color:#f5f5f5;">'
493- '<th style="text-align:left;padding:6px 10px;'
494- 'border-bottom:1px solid #e0e0e0;width:65px;">Severity</th>'
495- '<th style="text-align:left;padding:6px 10px;'
496- 'border-bottom:1px solid #e0e0e0;width:180px;">Parameter</th>'
497- '<th style="text-align:left;padding:6px 10px;'
498- 'border-bottom:1px solid #e0e0e0;">Details</th></tr>'
499- )
500- for idx , issue in enumerate (r .issues ):
501- bg = "#ffffff" if idx % 2 == 0 else "#fafafa"
502- if issue .severity == "ERROR" :
503- sev_html = (
504- '<span style="color:#D32F2F;font-weight:600;">'
505- '● ERROR</span>'
506- )
507- else :
508- sev_html = (
509- '<span style="color:#F57C00;font-weight:600;">'
510- '● WARN</span>'
488+ # --- Errors section ---
489+ if errors :
490+ parts .append (
491+ '<tr><td style="padding:8px 12px 4px 12px;">'
492+ '<strong style="font-size:12px;color:#D32F2F;">'
493+ '● Errors</strong></td></tr>'
494+ '<tr><td style="padding:0 12px;">'
495+ '<table role="presentation" width="100%" cellpadding="0"'
496+ ' cellspacing="0" style="font-size:12px;border:1px solid #f5c6cb;">'
497+ '<tr style="background-color:#FFEBEE;">'
498+ '<th style="text-align:left;padding:6px 10px;'
499+ 'border-bottom:1px solid #f5c6cb;width:180px;">Parameter</th>'
500+ '<th style="text-align:left;padding:6px 10px;'
501+ 'border-bottom:1px solid #f5c6cb;">Details</th></tr>'
502+ )
503+ for idx , issue in enumerate (errors ):
504+ bg = "#ffffff" if idx % 2 == 0 else "#fff5f5"
505+ parts .append (
506+ f'<tr style="background-color:{ bg } ;">'
507+ f'<td style="padding:5px 10px;border-bottom:1px solid #f5c6cb;'
508+ f'vertical-align:top;font-family:Consolas,monospace;'
509+ f'font-size:11px;word-break:break-all;">'
510+ f'{ _html_escape (issue .param_name )} </td>'
511+ f'<td style="padding:5px 10px;border-bottom:1px solid #f5c6cb;'
512+ f'vertical-align:top;">{ _html_escape (issue .message )} </td>'
513+ f'</tr>'
511514 )
515+ parts .append ("</table></td></tr>" )
516+
517+ # --- Warnings section ---
518+ if warnings :
512519 parts .append (
513- f'<tr style="background-color:{ bg } ;">'
514- f'<td style="padding:5px 10px;border-bottom:1px solid #eee;'
515- f'vertical-align:top;">{ sev_html } </td>'
516- f'<td style="padding:5px 10px;border-bottom:1px solid #eee;'
517- f'vertical-align:top;font-family:Consolas,monospace;'
518- f'font-size:11px;word-break:break-all;">'
519- f'{ _html_escape (issue .param_name )} </td>'
520- f'<td style="padding:5px 10px;border-bottom:1px solid #eee;'
521- f'vertical-align:top;">{ _html_escape (issue .message )} </td>'
522- f'</tr>'
520+ '<tr><td style="padding:8px 12px 4px 12px;">'
521+ '<strong style="font-size:12px;color:#F57C00;">'
522+ '● Warnings</strong></td></tr>'
523+ '<tr><td style="padding:0 12px 8px 12px;">'
524+ '<table role="presentation" width="100%" cellpadding="0"'
525+ ' cellspacing="0" style="font-size:12px;border:1px solid #ffe0b2;">'
526+ '<tr style="background-color:#FFF3E0;">'
527+ '<th style="text-align:left;padding:6px 10px;'
528+ 'border-bottom:1px solid #ffe0b2;width:180px;">Parameter</th>'
529+ '<th style="text-align:left;padding:6px 10px;'
530+ 'border-bottom:1px solid #ffe0b2;">Details</th></tr>'
523531 )
524- parts .append ("</table></td></tr>" )
532+ for idx , issue in enumerate (warnings ):
533+ bg = "#ffffff" if idx % 2 == 0 else "#fffaf0"
534+ parts .append (
535+ f'<tr style="background-color:{ bg } ;">'
536+ f'<td style="padding:5px 10px;border-bottom:1px solid #ffe0b2;'
537+ f'vertical-align:top;font-family:Consolas,monospace;'
538+ f'font-size:11px;word-break:break-all;">'
539+ f'{ _html_escape (issue .param_name )} </td>'
540+ f'<td style="padding:5px 10px;border-bottom:1px solid #ffe0b2;'
541+ f'vertical-align:top;">{ _html_escape (issue .message )} </td>'
542+ f'</tr>'
543+ )
544+ parts .append ("</table></td></tr>" )
525545 else :
526546 parts .append (
527547 '<tr><td style="padding:10px 12px;color:#2E7D32;'
0 commit comments