Dorian Villet

icon indicating a website link https://www.tipee.ch

icon location Switzerland, Lausanne icon location PHP technician and Symfony expert, I’m a code quality enthusiast at heart with over 15 years of professional experience.

Results 71 comments of Dorian Villet

local-global repositories (composer link command)

This issue is still a huge PITA, and neither https://github.com/gossi/composer-localdev-plugin nor https://github.com/franzliedke/studio (v0.13 and v0.14-alpha) currently work for me : I spent two days trying to get a proper setup...

Generic's sub-type is lost when passed through a callable

If by "special handling", you mean a custom DynamicReturnTypeExtension, then I'm confused, because I've created one too for our `map()` wrapper which extends PHPStan's (cf. code below), which I could...

Generic's sub-type is lost when passed through a callable

@rvanvelzen would it be feasible to make that "internal handling" pluggable ? Like being able to define a list of functions (and/or methods?) acting like array_map ?

Unresolvable type error in PHPUnit\assertSame when the same enum is passed for second time

Hey @rvanvelzen, I just encountered this issue as well, and managed to reproduce it in an as-small-as-possible-yet-fully-functional repository! :tada: https://github.com/gnutix/phpstan-assertsame-unresolvable-type-bug/pull/2/files

Unresolvable type error in PHPUnit\assertSame when the same enum is passed for second time

@rvanvelzen oh my, good catch... :scream: Adding `null` to the return type removes the inspection indeed. :+1:

Handle git external worktrees correctly

That's too bad. I would have enjoyed reinstalling lolcommits, but I mostly work in worktrees now. :(

Allow single line braces for empty function body

Very eager to see this one coming out too !

Allow single line braces for empty function body

Any chance this PR could get some attention ? Last review was almost 6 months ago.

Add LocalDateTimeRange

A comment worth referencing here : https://github.com/brick/date-time/issues/45#issuecomment-1173093592

Introduce a ZonedDateTimeRange

We've developed a [package](https://github.com/gammadia/date-time-extra/) on top of `brick/date-time` which includes a `LocalDateTimeInterval` object using an exclusive end. We use it quite extensively in our time-management project, and it works like...