4
97%
master: 97%

Ran 07 Jan 2020 09:19PM UTC

Files 157

Run time 8s

Badge

Embed ▾

Committed 06 Jan 2020 04:22AM UTC coverage: 98.959% (-0.007%) from 98.966%

Job # 2.7.0, gemfiles/rails_edge.gemfile

Build Type

push

travis-ci

Committed by

dblock

Commit Message

delete a reversible stackable values class (#1953)

* delete a reversible stackable values class

The reversible stackable values object was initialized for every endpoint,
however, it is only needed for keeping rescue handlers. The idea is
simple, handlers defined "closer" to an endpoint have higher priority.
That test https://github.com/ruby-grape/grape/blob/master/spec/grape/api_spec.rb#L3215-L3232
demonstrates how it works.

In our project rescue handlers are defined at the top level, so almost every
endpoint keeps the unused object.

The mentioned behavior is easy to achieve with the stackable values object and
the `reverse_each` method. Thus, endpoints keeps less objects and
have less code to be maintained.

Besides that, there are a few other simple performance optimizations.

Run Details

12828 of 12963 relevant lines covered (98.96%)

28.3 hits per line

ruby-grape / grape / 3722 / 4
97%
master: 97%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3722.4 (2.7.0, gemfiles/rails_edge.gemfile)

ruby-grape / grape / 3722 / 4 97% master: 97%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3722.4 (2.7.0, gemfiles/rails_edge.gemfile)

ruby-grape / grape / 3722 / 4
97%
master: 97%

README BADGES
x