@rafaelfranca We chose to use `tap` for two reasons:
1. `compute_and_store_digest` needs to always return the digest (the `if` would return nil if `caching?` is false)
2. It had already been used in a similar fashion in the same file (https://github.com/cloudspace/rails/blob/29f3f95bed0cfca5cffe1555a274ab13e6dd789c/actionview/lib/action_view/digestor.rb#L51-53)

Considering that, we can change it to not use `tap` if you would still rather it.