Bruce Weirdan

Results 291 comments of Bruce Weirdan

It broke again with latest php-integrator-base updates, it seems.

> can you tag this with "help wanted" and pin the issue? Done. I also added milestone that we can assign to issues / PRs.

> Enum are objects Can you elaborate? It seems Psalm already knows that: https://psalm.dev/r/a57335dce7?php=8.1

@kkmuffme yes. Some may be available in master only though, but most are already a part of some release.

@orklah there's already a number of packages removed on PHP 7.1, just add slevomat/coding-standard to that list. CS checks do not run on 7.1 anyway.

Basically we need a way for Psalm to distinguish purely asserting functions (those that exist only to assert things declared in `@psalm-assert`) and those that do something else as well...

re: stubs - how would that work? If I stub the Build target in the common file it would prevent all projects from building. Care to explain?

That's right, I don't build Soap extension because there's some kind of a conflict, specifics of which elude me at the moment. Here's relevant part of the commit message: >...

Could this be cherry-picked into current master? Things like proper PHP_OS value are useful, and that particular commit (kripper@4cb8a0c) applies cleanly even if the rest do not.

@VincentLanglet mind creating a reproducer in a separate repo?