Aidan Casey

@brendt to me that feels a little misleading/convoluted. If anything, I could see doing something like: ```env CACHE=true DISCOVERY_CACHE=${CACHE} ``` But even that feels slightly icky to me.

Yes, for sure we will want to utilize `clearstatcache`. 🙂

Hey, @innocenzi! See #272 where we discuss this a little bit. I'd like to keep this portion lightweight and independent of Flysystem, but potentially make use of it for the...

Definitely a good thought. I'll think about the API a bit. I'd like to also allow the disabling of that functionality if the developer determines they _don't_ want that to...

@brendt - What do you think about the API with the `Permission` class here? I have a couple of thoughts... We could simplify the combinations by allowing an integer to...

@brendt - Good thought. Instead of counting back from 777, I am counting back from the current value. It feels safer to me than something like: ```php // With counting...

Mental note: - Exception messages are a little inconsistent right now. Clean that up. - Method behavior is a little inconsistent right now. Clean that up. - Determine a good...

@brendt - I have some thoughts here... While less developer friendly, have you tried parsing files with `nikic/php-parser` at all? I'd imagine it'd be faster, but unsure. We'd also avoid...

> Taking around a minute to warm the cache is pretty significant It didn't. 1 minute = 60 seconds. The longest number here is 0.74s which is 740 milliseconds or...

> I guess that's not to warm the cache then and it is just to pass thought the checks after the cache is already there. I'd suggest taking another look...