Dan Singerman

I think this is a good change. I like having the `cache` option, as I can imagine there may be cases where it's harder to know when to invalidate the...

I think if it's on by default, then it's not obvious there's caching going on. From a developer experience point of view I'd favour having it off by default, but...

Adding caching to the `parent_ids_for` method means when the ability_cache needs to be created it's going to do DB update for each unique call to `parent_ids_for`. Instead, how about adding...

Yes I think you're right. We'd need that still especially in the case you had a permit statement with `cache: false` followed by a permit statement with `cache: true`. In...

> @dansingerman after merging #21, using `cache: true` as the key here feels confusing. > > Now that we've introduced caching at two different levels, I think we need to...