17 pragma based router

[sebastianconcept]

The builder now generates the methods that defines routes in the presenters' class side.

Feels quite comfortable and makes the a CRUD fully usable by default just after being created without any additional change.

[paulwilke]

Checked out the PR and wanted to drop some quick thoughts. First off, love the router revamp and the use of pragma for routing – neat and flexible without being bloated.

The handling of dev vs. production environments is a smart move for maintainability. Might be worth adding a few comments or brief docs for clarity, especially for us not knee-deep in the details.

Lastly, while the optimizations and refactoring are top-notch, keeping an eye on test coverage would be cool, just to ensure no unintended breakages sneak through

[sebastianconcept]

Thank you for taking a look @paulwilke ! I've added this one to have a reminder on having a how-to for deploys
#21

And for detecting regressions and gaining trust improve that coverage is a must. Seeing that 1% is painful 🔥 but it can start with the router. I push a commit for that to break the ice on this.

[codecov]

Codecov Report

Attention: Patch coverage is 18.96552% with 47 lines in your changes are missing coverage. Please review.

Project coverage is 4.93%. Comparing base (78cff44) to head (51376ac).

Files	Patch %	Lines
Ride/RideServer.class.st	32.35%	23 Missing ⚠️
Ride/RideResource.class.st	0.00%	16 Missing ⚠️
Ride/Ride.class.st	0.00%	8 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff            @@
##           master     #19      +/-   ##
=========================================
+ Coverage    0.71%   4.93%   +4.21%     
=========================================
  Files          25      25              
  Lines        3070    3123      +53     
=========================================
+ Hits           22     154     +132     
+ Misses       3048    2969      -79     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.