Skip to content

Reduce routes memory footprint#25

Merged
Baptouuuu merged 1 commit intodevelopfrom
improve-routes-memory
Apr 12, 2026
Merged

Reduce routes memory footprint#25
Baptouuuu merged 1 commit intodevelopfrom
improve-routes-memory

Conversation

@Baptouuuu
Copy link
Copy Markdown
Member

Instead of using a lazy Sequence from the start, that creates a new generator for each new route, a lazy Sequence built from the in memory one is built just before calling each handler.

This way handlers declared after the matched route are still not called, while using a single array to store all the callables (instead of n generators).

@Baptouuuu Baptouuuu self-assigned this Apr 12, 2026
@Baptouuuu Baptouuuu added the enhancement New feature or request label Apr 12, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.51%. Comparing base (c215097) to head (3016ec5).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files
@@              Coverage Diff              @@
##             develop      #25      +/-   ##
=============================================
+ Coverage      94.50%   95.51%   +1.01%     
  Complexity       129      129              
=============================================
  Files             14       14              
  Lines           1583     1583              
=============================================
+ Hits            1496     1512      +16     
+ Misses            87       71      -16     

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

@Baptouuuu Baptouuuu merged commit cff67a3 into develop Apr 12, 2026
40 checks passed
@Baptouuuu Baptouuuu deleted the improve-routes-memory branch April 12, 2026 09:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant