Fix MAGI Medicaid household construction#8169
Fix MAGI Medicaid household construction#8169daphnehanse11 wants to merge 4 commits intoPolicyEngine:mainfrom
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8169 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 7 16 +9
Lines 215 224 +9
Branches 6 2 -4
=========================================
+ Hits 215 224 +9
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Initial read-only review via gh/PR diff before edits. What I checked:
Findings I am patching in a fresh /tmp worktree:
I do not see a broader blocker in the submitted diff yet; these look like focused PolicyEngine-style fixes before merge. |
|
Follow-up after the initial patch: Codecov was still flagging missing patch coverage in Local verification:
Pushed follow-up commit: |
Fixes #5907
What this does
Known gaps
Validation
uv run --with ruff ruff format .uv run --with ruff ruff check .uv run python -m policyengine_core.scripts.policyengine_command test policyengine_us/tests/policy/baseline/gov/hhs/medicaid/income/medicaid_uses_non_filer_rules.yaml policyengine_us/tests/policy/baseline/gov/hhs/medicaid/income/medicaid_income_level.yaml policyengine_us/tests/policy/baseline/gov/hhs/medicaid/income/tax_unit_medicaid_income_level.yaml policyengine_us/tests/policy/baseline/gov/hhs/chip policyengine_us/tests/policy/baseline/gov/aca/eligibility/aca_magi.yaml policyengine_us/tests/policy/baseline/gov/aca/ptc/integration.yaml policyengine_us/tests/policy/baseline/gov/aca/ptc/separate_esi.yaml -c policyengine_us