Seems OK, however obscure if you don't have the context when reading this code. I *think* removing :all would have been clearer. Up to you if you want to try doing that or add a comment that `error` can be `:all`.