Skip to content

[php 8.3] Add json_validate rule#7213

Merged
samsonasik merged 1 commit intorectorphp:mainfrom
arshidkv12:json_validate_rule
Sep 4, 2025
Merged

[php 8.3] Add json_validate rule#7213
samsonasik merged 1 commit intorectorphp:mainfrom
arshidkv12:json_validate_rule

Conversation

@arshidkv12
Copy link
Contributor

No description provided.

* @see https://wiki.php.net/rfc/deprecations_php_8_5#deprecate_the_sleep_and_wakeup_magic_methods
* @see \Rector\Tests\Php85\Rector\MethodCall\SleepToSerializeRector\SleepToSerializeRectorTest
*/
final class SleepToSerializeRector extends AbstractRector implements MinPhpVersionInterface
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you seems merge 2 PRs into single branch, use rebase to works on multiple feature branch, I created a blog post about it:

https://samsonasik.wordpress.com/2015/09/16/practical-git-4-rebasing-conflicted-task-branch-against-primary-branch/

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you

Copy link
Member

@samsonasik samsonasik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good, let's try in the wild :)

@samsonasik samsonasik merged commit a72065d into rectorphp:main Sep 4, 2025
49 checks passed
@samsonasik
Copy link
Member

Thank you @arshidkv12

arshidkv12 added a commit to arshidkv12/rector-src that referenced this pull request Sep 4, 2025
Co-authored-by: Arshid <arshid@Arshids-MacBook-Air.local>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 8, 2026

This pull request has been automatically locked because it has been closed for 150 days. Please open a new PR if you want to continue the work.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Feb 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants