Skip to content

Feat: add the fmt_index() method - #888

Merged
rich-iannone merged 7 commits into
mainfrom
feat-fmt-index
Sep 24, 2026
Merged

rich-iannone merged 7 commits into
mainfrom
feat-fmt-index

Conversation

@rich-iannone

Copy link
Copy Markdown
Member

This PR introduces a new "index" formatting feature that allows users to format numeric values as index characters (e.g., 1 → "A", 27 → "AA") in both the GT and vals APIs (as fmt_index()). The implementation includes support for both "repeat" and "excel" style algorithms, case selection, and custom patterns.

@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 93.75000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 98.34%. Comparing base (284d28a) to head (6289e40).

Files with missing lines Patch % Lines
great_tables/_formats.py 92.68% 3 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #888      +/-   ##
==========================================
- Coverage   98.36%   98.34%   -0.03%     
==========================================
  Files          52       52              
  Lines        8152     8200      +48     
==========================================
+ Hits         8019     8064      +45     
- Misses        133      136       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@rich-iannone
rich-iannone merged commit 331d373 into main Sep 24, 2026
13 of 15 checks passed
@rich-iannone
rich-iannone deleted the feat-fmt-index branch September 24, 2026 23:55

This branch was successfully deployed

1 active deployment
pr-888 — 6289e406 Deployed Sep 24, 2026 by github-actions[bot]
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