Skip to content

Remove the routes only the legacy site used, check health at /health, and stop serving Canada, Nigeria and Israel - #4017

Draft
anth-volk wants to merge 5 commits into
masterfrom
v2plan/1g-128-remove-unused-routes
Draft

anth-volk wants to merge 5 commits into
masterfrom
v2plan/1g-128-remove-unused-routes

Require the dev extra in the cross-database integration workflow test

b548822
Select commit
Loading
Failed to load commit list.
Codecov / codecov/project failed Oct 30, 2026 in 1s

51.45% (-36.67%) compared to 424cf47

View this Pull Request on Codecov

51.45% (-36.67%) compared to 424cf47

Details

Codecov Report

❌ Patch coverage is 69.23077% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 51.45%. Comparing base (424cf47) to head (b548822).
⚠️ Report is 12 commits behind head on master.

Files with missing lines Patch % Lines
policyengine_api/country.py 0.00% 2 Missing ⚠️
policyengine_api/fastapi_routes/health.py 71.42% 2 Missing ⚠️

❗ There is a different number of reports uploaded between BASE (424cf47) and HEAD (b548822). Click for more details.

HEAD has 1 upload less than BASE
Flag BASE (424cf47) HEAD (b548822)
2 1
Additional details and impacted files
@@             Coverage Diff             @@
##           master    #4017       +/-   ##
===========================================
- Coverage   88.11%   51.45%   -36.67%     
===========================================
  Files         285      267       -18     
  Lines       16577    15026     -1551     
  Branches     2712     2399      -313     
===========================================
- Hits        14607     7731     -6876     
- Misses       1292     6656     +5364     
+ Partials      678      639       -39     

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