Skip to content

add Printf support#201

Merged
hyrodium merged 2 commits intoJuliaMath:masterfrom
aplavin:printf
Oct 6, 2025
Merged

add Printf support#201
hyrodium merged 2 commits intoJuliaMath:masterfrom
aplavin:printf

Conversation

@aplavin
Copy link
Contributor

@aplavin aplavin commented Jul 30, 2025

It's very convenient to print intervals (and other objects) with flexible number formatting. Printf is reasonably extensible, and this PR adds Printf support for intervals. + corresponding tests.
FWIW, there are a few packages with this kind of Printf support already: https://juliahub.com/ui/Search?type=code&q=Printf.fmt.

@codecov
Copy link

codecov bot commented Jul 30, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 98.21%. Comparing base (13c76e1) to head (53c8489).
⚠️ Report is 22 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
- Coverage   99.11%   98.21%   -0.90%     
==========================================
  Files           6        7       +1     
  Lines         225      280      +55     
==========================================
+ Hits          223      275      +52     
- Misses          2        5       +3     

☔ View full report in Codecov by Sentry.
📢 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.

@aplavin
Copy link
Contributor Author

aplavin commented Jul 31, 2025

Not sure what codecov job complains about, all new code seems covered

@aplavin
Copy link
Contributor Author

aplavin commented Aug 21, 2025

bump...

@aplavin
Copy link
Contributor Author

aplavin commented Sep 3, 2025

another gentle bump...
can this approved PR be merged?

@aplavin
Copy link
Contributor Author

aplavin commented Oct 5, 2025

anybody?..

Copy link
Collaborator

@hyrodium hyrodium left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the late review. LGTM!

@hyrodium hyrodium merged commit 450defe into JuliaMath:master Oct 6, 2025
12 of 13 checks passed
@aplavin aplavin deleted the printf branch November 6, 2025 20:01
@aplavin
Copy link
Contributor Author

aplavin commented Nov 6, 2025

can we get a release with this? :) @hyrodium

@hyrodium hyrodium mentioned this pull request Nov 7, 2025
bvdmitri added a commit to bvdmitri/IntervalSets.jl that referenced this pull request Nov 14, 2025
dlfivefifty pushed a commit that referenced this pull request Nov 19, 2025
* Revert "add Printf support (#201)"

This reverts commit 450defe.

* Bump version from 0.7.12 to 0.7.13
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.

3 participants