Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

[Php81] Skip as Arg of non-native function call on FunctionLikeToFirstClassCallableRector#7592

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
samsonasik merged 4 commits intomainfromskip-as-arg-of-non-native-function
Oct 27, 2025

Conversation

@samsonasik
Copy link
Member

@samsonasiksamsonasik commentedOct 27, 2025•
edited
Loading

When on method call, it can be anything defined in caller, this PR skip if it is as arg of non-native function call.

The skipped parts already in other fixtures, the updated fixture for demonstrate trailing comma :)

@samsonasik
Copy link
MemberAuthor

@maks-oleksyuk this should cover most of the issue it have, it skip if it used as arg of non-native function call :)

maks-oleksyuk reacted with eyes emoji

@samsonasik
Copy link
MemberAuthor

All checks have passed 🎉@TomasVotruba I am merging it ;)

TomasVotruba reacted with thumbs up emoji

@samsonasiksamsonasikenabled auto-merge (squash)October 27, 2025 15:49
@samsonasiksamsonasik merged commit0bbb4c7 intomainOct 27, 2025
53 checks passed
@samsonasiksamsonasik deleted the skip-as-arg-of-non-native-function branchOctober 27, 2025 15:50
@maks-oleksyuk
Copy link

@maks-oleksyuk this should cover most of the issue it have, it skip if it used as arg of non-native function call :)

Yes, I checked it on my own case, and indeed, it no longer causes the error I described earlier. Thank you.

samsonasik reacted with thumbs up emoji

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@samsonasik@maks-oleksyuk@actions-user
[8]ページ先頭
©2009-2025 Movatter.jp