Skip to content

Release v2.9.0#270

Merged
igerber merged 1 commit intomainfrom
release/v2.9.0
Apr 5, 2026
Merged

Release v2.9.0#270
igerber merged 1 commit intomainfrom
release/v2.9.0

Conversation

@igerber
Copy link
Copy Markdown
Owner

@igerber igerber commented Apr 5, 2026

Summary

  • Bump version to 2.9.0 across all version locations (pyproject.toml, init.py, Cargo.toml, llms-full.txt)
  • Add CHANGELOG entry for v2.9.0 covering: WooldridgeDiD (ETWFE), EfficientDiD survey+covariates, survey real-data validation, label-gated CI, documentation dependency map, and associated fixes

Methodology references

  • N/A — version bump only, no methodology changes in this PR

Validation

  • No test changes — release bookkeeping only

Security / privacy

  • Confirm no secrets/PII in this PR: Yes

Generated with Claude Code

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 5, 2026

Overall Assessment

Looks good — no unmitigated P0/P1 findings. This PR is release bookkeeping plus changelog text; it does not modify estimator code, weighting, variance/SE computation, identification checks, or defaults.

Executive Summary

Methodology

  • Severity: P3; Impact: CHANGELOG.md:19 says both survey-validation differences are “documented in REGISTRY.md,” but only the subpopulation df case is explicitly labeled there at docs/methodology/REGISTRY.md:2472. The TWFE item is only indirectly supported by the absorbed-FE methodology notes at docs/methodology/REGISTRY.md:240 and docs/methodology/REGISTRY.md:2273. This is documentation clarity only; no estimator or inference logic changes in this PR. Concrete fix: either add an explicit TWFE survey-validation note to REGISTRY.md or soften the changelog wording to say the differences are documented across the methodology docs.

Code Quality

Performance

  • No findings. No runtime or hot-path code changed.

Maintainability

  • No findings. Version display in the built docs is already derived from package metadata via docs/conf.py:19, which reduces release-bump drift.

Tech Debt

  • Severity: P3 (tracked in TODO.md); Impact: this PR touches docs/llms-full.txt, but .txt AI guides are still outside snippet CI coverage, which is already tracked at TODO.md:100. This is mitigated per the review policy. Concrete fix: none required for approval; the existing TODO item is sufficient tracking.

Security

  • No findings. The diff is metadata/docs only and does not add secret-handling, dependency, network, or execution surface.

Documentation/Tests

  • Severity: P3; Impact: the roadmap still presents v2.8.4 as the current status at ROADMAP.md:11, while this PR advances release metadata to 2.9.0. That creates minor user-facing version inconsistency across docs. Concrete fix: update the hard-coded version in ROADMAP.md, or remove the version number from that sentence so future release PRs do not need to keep it in sync.
  • No test blocker. This PR does not change executable estimator behavior, so the absence of new methodology tests is acceptable for a release-bookkeeping PR.

@igerber igerber added the ready-for-ci Triggers CI test workflows label Apr 5, 2026
@igerber igerber merged commit 63f51f0 into main Apr 5, 2026
21 of 23 checks passed
@igerber igerber deleted the release/v2.9.0 branch April 5, 2026 01:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-for-ci Triggers CI test workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant