Ruud Kamphuis

Results 363 comments of Ruud Kamphuis

It seems that the RFC is accepted and can be used, see this comment https://github.com/graphql/graphql-spec/pull/825/#issuecomment-2945309875

@fabpot What do you think of this idea? Today I had again a situation where this would have helped tremendously by doing something like `{% if app.user.account.id == some.other.id %}`...

I guess this can be solved in user land via scripting. If anyone's interested: ```bash #!/bin/bash BRANCHPOINT=$(git town config get-parent) BRANCH=$(git branch --show-current) COMMITS=$(git log --no-merges --oneline --no-decorate "$BRANCHPOINT".."$BRANCH" |...

Yeah I also do this a few times a week. I work on a feature and see some low hanging fruit that I fix in separate commits. Those I want...

@Omranic Thanks for this amazing library. What's needed to get this one merged? Any help needed?

@Omranic Thanks ⚡ Would you be able to tag it?

@nicolas-grekas I noticed this is not tagged yet. Could that be done 🙏 ?

Also interesting, as soon as you remove the pass-by-reference, it works better: https://phpstan.org/r/ec47dbd1-03ed-48ec-b211-b926845bf0c3

I found the cause of this problem. It's because the types get generalized: https://github.com/phpstan/phpstan-src/blame/df39688b3849004f61fc4ed83ee4c1e6e3b3399d/src/Analyser/MutatingScope.php#L4859-L4865 I wonder why this is needed?