Skip to content

fix(formatters): pad priority labels for consistent TRIAGE column alignment#449

Merged
MathurAditya724 merged 1 commit intomainfrom
fix/quick-padding
Mar 17, 2026
Merged

fix(formatters): pad priority labels for consistent TRIAGE column alignment#449
MathurAditya724 merged 1 commit intomainfrom
fix/quick-padding

Conversation

@MathurAditya724
Copy link
Member

Summary

  • Pads Med and Low priority labels to 4 characters (matching High) in the TRIAGE column of issue list output, so the fixability percentages align consistently across rows.

Before

TRIAGE
High 82%
Med 65%
Low 40%

After

TRIAGE
High 82%
Med  65%
Low  40%

The padding is applied inside the colorTag() call in formatPriorityLabel(), so string-width correctly counts the visible width for column sizing. Critical (8 chars) is left as-is since it's visually distinct.

@github-actions
Copy link
Contributor

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (dashboard) Add dashboard list, view, and create commands by betegon in #406
  • Add sentry schema command for API introspection by BYK in #437

Bug Fixes 🐛

  • (dsn) Prevent hang during DSN auto-detection in repos with test fixtures by BYK in #445
  • (formatters) Pad priority labels for consistent TRIAGE column alignment by MathurAditya724 in #449
  • (upgrade) Remove hard chain depth cap for nightly delta upgrades by BYK in #444

Internal Changes 🔧

  • Cache org listing in listOrganizations + DSN shortcut for issue view by betegon in #446

🤖 This preview updates automatically when you update the PR.

@MathurAditya724 MathurAditya724 enabled auto-merge (squash) March 17, 2026 16:55
@github-actions
Copy link
Contributor

Codecov Results 📊

111 passed | Total: 111 | Pass Rate: 100% | Execution Time: 0ms

📊 Comparison with Base Branch

Metric Change
Total Tests
Passed Tests
Failed Tests
Skipped Tests

✨ No test changes detected

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 1107 uncovered lines.
✅ Project coverage is 95.26%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
human.ts 96.37% ⚠️ 45 Missing
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    95.24%    95.26%    +0.02%
==========================================
  Files          174       174         —
  Lines        23276     23342       +66
  Branches         0         0         —
==========================================
+ Hits         22168     22235       +67
- Misses        1108      1107        -1
- Partials         0         0         —

Generated by Codecov Action

@MathurAditya724 MathurAditya724 merged commit b71258d into main Mar 17, 2026
21 checks passed
@MathurAditya724 MathurAditya724 deleted the fix/quick-padding branch March 17, 2026 16:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant