Results 379 comments of Chris Brown

> language selector is hidden when editing a file I believe the understanding was that if you have the file in edit mode, you probably have changes to save. Once...

While yes it's a string because it comes from the database as a string, the "string"'s content is always expected to be an integer (never float, never alpha, never symbols)....

Correct. That's why it's not typehinted. The docblock says `int` because we intend to use that parameter as an integer value. And it's not typehinted as such because we know...

Unofficial policy: - typehints are preferable, and no need for docblock if all parameters are typehinted - if a typehint cannot be used, then a docblock is desireable if the...

> If a parameter cannot be type-hinted, is that not an indication that the various calling code chunks should be reviewed/rationalised to pass the same type? Where possible. Yes. Or...

Hi. Still considering this for inclusion in next major release. A few things enum-related will be breaking changes, so queuing them for then. Re-opening.

Note to self, during future review: `BackedEnum` is not supported before PHP 8.1 ... which is why the PHP 8.0 tests are failing. Hence another reason for queuing for a...

@freekmurze - Can you investigate the change of display regarding blockquotes in the docs

Another example

Thanks for the detailed report. I'm surprised that the `collect()->map()` call on line 69 would cause a memory leak. How many `auth.guard` array entries do you have (both in your...