Skip to content

Make doctests independent of Distributions show output - #688

Merged
bvdmitri merged 3 commits into
mainfrom
fix/distributions-0.25.133-doctests
Nov 3, 2026
Merged

bvdmitri merged 3 commits into
mainfrom
fix/distributions-0.25.133-doctests

Conversation

@bvdmitri

@bvdmitri bvdmitri commented Nov 3, 2026

Copy link
Copy Markdown
Member

Distributions v0.25.133 changed how distributions print (JuliaStats/Distributions.jl#2092), e.g. Gamma(10.0, 2.0) instead of Distributions.Gamma{Float64}(α=10.0, θ=2.0). This broke the Message/Marginal doctests (seen in ExponentialFamily.jl#328 downstream CI).

The doctests now compare the wrapped distribution with == instead of matching printed output, so they pass on both old and new Distributions.

🤖 Generated with Claude Code

Distributions v0.25.133 changed how distributions print
(JuliaStats/Distributions.jl#2092).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@bvdmitri
bvdmitri requested a review from Nimrais November 3, 2026 15:07
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013ZGXE7rsjq3rYesnaJfNof
@codecov

codecov Bot commented Nov 3, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.22%. Comparing base (007cfd1) to head (64ef48a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #688   +/-   ##
=======================================
  Coverage   93.22%   93.22%           
=======================================
  Files          31       31           
  Lines        1785     1785           
=======================================
  Hits         1664     1664           
  Misses        121      121           

☔ 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.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013ZGXE7rsjq3rYesnaJfNof
@bvdmitri
bvdmitri merged commit ee7f5b5 into main Nov 3, 2026
7 of 8 checks passed
@bvdmitri
bvdmitri deleted the fix/distributions-0.25.133-doctests branch November 3, 2026 21:25
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.

2 participants