From ae8dcf215773813ad69458dfa3e8456a8261ad60 Mon Sep 17 00:00:00 2001
From: Junya Morioka <mjun@mjunya.com>
Date: Wed, 28 Jan 2026 18:35:55 +0900
Subject: [PATCH] refactor: move coverage to Packages subsection and remove
 excluded column

Move coverage section under ## Packages as ### Coverage subsection.
Remove the Excluded column from the coverage table for simplicity.
---
 README.md                 | 18 +++++++++---------
 update_readme_coverage.py | 12 +++++-------
 2 files changed, 14 insertions(+), 16 deletions(-)

diff --git a/README.md b/README.md
index 9aede09..9b9e656 100644
--- a/README.md
+++ b/README.md
@@ -38,23 +38,23 @@ wget https://github.com/mjun0812/flash-attention-prebuild-wheels/releases/downlo
 pip install ./flash_attn-2.6.3+cu124torch2.5-cp312-cp312-linux_x86_64.whl
 ```
 
+## Packages
+
 <!-- COVERAGE_START -->
-## Package Coverage
+### Coverage
 
 ![Linux x86_64](https://img.shields.io/badge/Linux_x86_64-83.0%25-yellow?style=for-the-badge)
 ![Linux ARM64](https://img.shields.io/badge/Linux_ARM64-70.0%25-yellow?style=for-the-badge)
 ![Windows](https://img.shields.io/badge/Windows-60.0%25-red?style=for-the-badge)
 
-| Platform | Existing | Missing | Excluded | Coverage |
-|----------|----------|---------|----------|----------|
-| Linux x86_64 | 142 | 29 | 189 | 83.0% |
-| Linux ARM64 | 21 | 9 | 0 | 70.0% |
-| Windows | 18 | 12 | 0 | 60.0% |
-| **Total** | **181** | **50** | **189** | **78.4%** |
+| Platform | Existing | Missing | Coverage |
+|----------|----------|---------|----------|
+| Linux x86_64 | 142 | 29 | 83.0% |
+| Linux ARM64 | 21 | 9 | 70.0% |
+| Windows | 18 | 12 | 60.0% |
+| **Total** | **181** | **50** | **78.4%** |
 <!-- COVERAGE_END -->
 
-## Packages
-
 > [!NOTE]
 > Since v0.7.0, wheels are built with manylinux2_28 platform.
 > These wheels for Linux x86_64 and ManyLinux are compatible with old glibc versions (<=2.17).
diff --git a/update_readme_coverage.py b/update_readme_coverage.py
index 72fa9bb..a8129fd 100644
--- a/update_readme_coverage.py
+++ b/update_readme_coverage.py
@@ -80,7 +80,7 @@ def make_badge_url(label: str, coverage_pct: float) -> str:
 
 def generate_coverage_markdown(stats_by_platform: dict[str, dict]) -> str:
     """Generate the coverage markdown block."""
-    lines = [COVERAGE_START, "## Package Coverage", ""]
+    lines = [COVERAGE_START, "### Coverage", ""]
 
     # Badges
     for platform_key, display_name in PLATFORMS.items():
@@ -94,12 +94,11 @@ def generate_coverage_markdown(stats_by_platform: dict[str, dict]) -> str:
     lines.append("")
 
     # Table
-    lines.append("| Platform | Existing | Missing | Excluded | Coverage |")
-    lines.append("|----------|----------|---------|----------|----------|")
+    lines.append("| Platform | Existing | Missing | Coverage |")
+    lines.append("|----------|----------|---------|----------|")
 
     total_existing = 0
     total_missing = 0
-    total_excluded = 0
 
     for platform_key, display_name in PLATFORMS.items():
         s = stats_by_platform.get(platform_key)
@@ -108,18 +107,17 @@ def generate_coverage_markdown(stats_by_platform: dict[str, dict]) -> str:
         total = s["existing"] + s["missing"]
         pct = f"{s['existing'] / total * 100:.1f}%" if total > 0 else "N/A"
         lines.append(
-            f"| {display_name} | {s['existing']} | {s['missing']} | {s['excluded']} | {pct} |"
+            f"| {display_name} | {s['existing']} | {s['missing']} | {pct} |"
         )
         total_existing += s["existing"]
         total_missing += s["missing"]
-        total_excluded += s["excluded"]
 
     grand_total = total_existing + total_missing
     grand_pct = (
         f"{total_existing / grand_total * 100:.1f}%" if grand_total > 0 else "N/A"
     )
     lines.append(
-        f"| **Total** | **{total_existing}** | **{total_missing}** | **{total_excluded}** | **{grand_pct}** |"
+        f"| **Total** | **{total_existing}** | **{total_missing}** | **{grand_pct}** |"
     )
     lines.append(COVERAGE_END)