Larry Garfield

Documentation unclear/inconsistent about rule names

3

## Expected Behavior On the rules list documentation page (https://pinterest.github.io/ktlint/latest/rules/standard), there is a great deal of inconsistency in how rules are named. For example, trailing commas. https://pinterest.github.io/ktlint/latest/rules/standard/#trailing-comma-on-call-site The "configuration settings"...

Resolves #98. This seems the obvious way to do it. I debated the RECOMMENDED part. I'm open to changing that if other WG members feel strongly about it. WG members...

Clarify that php open tags must be lowercase

1

Resolves #91. WG members (only): Please 👍 this thread to indicate your approval, or comment to the contrary.

Resolves #95 This is largely just a direct copy from php-src, as planned. All I did was make the "shoulds" capitalized. WG members (only): Please 👍 this thread to indicate...

The Road to 2.1

3

A new PHP release is coming soon, so it's time to get down and catch up on PER-CS! My expectation is that the next release will be 2.1, not 3.0...

Replaces #83 PHP Internals has adopted guidelines for when and how to handle capitalization of abbreviations in class/method names. We should follow suit and just steal their language directly, and/or...

PSR-13 support

4

I know this library is intended to be a "pure" implementation of PSR-7 and its related PSRs. However, that means it also supports PSR-15 and PSR-17. What about PSR-13? PSR-13...

getBasePath() returns incorrect value for bare stream wrapper path

2

I've run into a bug where trying to glob a stream path on just the root fails. Specifically, in my app, `Glob::getBasePath('foo://');` returns `foo:/`, rather than `foo://`. That means when...

Incorrect regex generated for patterns with **

1

I'm trying to filter some paths, where I want to use **. Unfortunately, the regex that's getting computed internal to `Glob::filter()` seems to be incorrect. For example, the pattern `/**`...

## Detailed description Currently, Serde tries to use the default value from a matching-name constructor parameter as a default for a property. That's fine if it's a promoted property. If...