Skip to content

Skip StrvalToType for strval first class callable syntax#4046

Merged
TomasVotruba merged 2 commits into
rectorphp:mainfrom
NicoImparfait:skip-strval-to-type-on-first-class-callable
Jun 2, 2023
Merged

Skip StrvalToType for strval first class callable syntax#4046
TomasVotruba merged 2 commits into
rectorphp:mainfrom
NicoImparfait:skip-strval-to-type-on-first-class-callable

Conversation

@NicoImparfait

@NicoImparfait NicoImparfait commented Jun 2, 2023

Copy link
Copy Markdown
Contributor

@NicoImparfait NicoImparfait force-pushed the skip-strval-to-type-on-first-class-callable branch from 993f172 to e6dbb57 Compare June 2, 2023 09:44
@NicoImparfait NicoImparfait force-pushed the skip-strval-to-type-on-first-class-callable branch from e6dbb57 to e3243c0 Compare June 2, 2023 09:47
@samsonasik

Copy link
Copy Markdown
Member

Rebase seems needed

@NicoImparfait NicoImparfait force-pushed the skip-strval-to-type-on-first-class-callable branch from 21db869 to 2effb90 Compare June 2, 2023 09:58
@NicoImparfait NicoImparfait marked this pull request as ready for review June 2, 2023 09:58
@samsonasik

Copy link
Copy Markdown
Member

Test error seems unrelated, we can wait until all green on main branch first, then rebase once more :)

@TomasVotruba

Copy link
Copy Markdown
Member

Looks good 👍 Let's ship it :)

@TomasVotruba TomasVotruba merged commit b8a0e84 into rectorphp:main Jun 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

StrvalToTypeCastRector crash with variadic param

3 participants