• Home
  • Features
  • Pricing
  • Docs
  • Announcements
  • Sign In

ruby-grape / grape / 26744664330 / 23
97%
master: 97%

Build:
Build:
LAST BUILD BRANCH: chore/raise-dependency-lower-bounds
DEFAULT BRANCH: master
Ran 01 Jun 2026 08:48AM UTC
Files 165
Run time 7s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

01 Jun 2026 08:46AM UTC coverage: 95.623% (+0.004%) from 95.619%
26744664330.23

Pull #2754

github

ericproulx
Merge routing args in place in Router#process_route

process_route built `args = env[GRAPE_ROUTING_ARGS] || { route_info: route }`
and then reassigned `env[GRAPE_ROUTING_ARGS] = args.merge(route_params)`, which
allocates a second Hash for every matched route that carries params.

Set the routing-args Hash once with `||=` and `merge!` the route params into it
in place, dropping the extra allocation. The resulting routing args are
identical (same keys and values); only the merge is now in-place. `route_params`
is a freshly built Hash, so nothing shared is mutated.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Pull Request #2754: Merge routing args in place in Router#process_route

1078 of 1193 branches covered (90.36%)

Branch coverage included in aggregate %.

3488 of 3582 relevant lines covered (97.38%)

971.78 hits per line

Source Files on job run-3.3-gemfiles/rack_2_2.gemfile - 26744664330.23
  • Tree
  • List 165
  • Changed 2
  • Source Changed 2
  • Coverage Changed 1
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses
  • Back to Build 26744664330
  • 415b6d38 on github
  • Prev Job for on perf/routing-args-in-place-merge (#26681995871.7)
  • Next Job for on perf/routing-args-in-place-merge (#26754696924.21)
STATUS · Troubleshooting · Open an Issue · Sales · Support · CAREERS · ENTERPRISE · START FREE · SCHEDULE DEMO
ANNOUNCEMENTS · TWITTER · TOS & SLA · Supported CI Services · What's a CI service? · Automated Testing

© 2026 Coveralls, Inc