Skip to content

add JSON_THROW_ON_ERROR even if there is already a flag#7867

Merged
samsonasik merged 11 commits intorectorphp:mainfrom
Lenny4:main
Feb 4, 2026
Merged

add JSON_THROW_ON_ERROR even if there is already a flag#7867
samsonasik merged 11 commits intorectorphp:mainfrom
Lenny4:main

Conversation

@Lenny4
Copy link
Contributor

@Lenny4 Lenny4 commented Jan 30, 2026

@Lenny4
Copy link
Contributor Author

Lenny4 commented Jan 30, 2026

ready

…d use ! instanceof check instead of ! is_null
Lenny4 and others added 2 commits February 4, 2026 03:07
Co-authored-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
Co-authored-by: Abdul Malik Ikhsan <samsonasik@gmail.com>
@Lenny4
Copy link
Contributor Author

Lenny4 commented Feb 4, 2026

@samsonasik done

@samsonasik
Copy link
Member

Could you rebase latest main branch so CI can all green? Thank you.

@Lenny4
Copy link
Contributor Author

Lenny4 commented Feb 4, 2026

@samsonasik done

@samsonasik
Copy link
Member

Looks good to me, @TomasVotruba let's merge ;)

Thank you @Lenny4

@samsonasik samsonasik merged commit eafa741 into rectorphp:main Feb 4, 2026
59 checks passed
@TomasVotruba
Copy link
Member

TomasVotruba commented Feb 4, 2026

Thank you @Lenny4 👍

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Incorrect behavior of JsonThrowOnErrorRector

3 participants