Skip to content
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 our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert all is* changes #3584

Merged
merged 7 commits into from
Apr 8, 2023
Merged

Revert all is* changes #3584

merged 7 commits into from
Apr 8, 2023

Conversation

samsonasik
Copy link
Member

@TomasVotruba @yguedidi let's try revert all is* changes.

@samsonasik
Copy link
Member Author

It seems working ok now, let's merge 👍

@samsonasik samsonasik merged commit dbb0e19 into main Apr 8, 2023
@samsonasik samsonasik deleted the revert-back branch April 8, 2023 13:19
@TomasVotruba
Copy link
Member

Thanks 👍

@yguedidi
Copy link
Contributor

yguedidi commented Apr 8, 2023

Thank you @samsonasik and again sorry for the extra work!
So if I understood correctly both tests and rector run on rector must be green to be OK?
I'll opens some PRs again! :)

@samsonasik
Copy link
Member Author

I already start with most safe type: void #3586

You can see the rector and cs run on the workflow https://github.com/rectorphp/rector-src/blob/main/.github/workflows/rector.yaml#L12

You can run:

bin/rector src tests rules-tests packages packages-tests
bin/rector rules
 bin/rector config utils

@yguedidi
Copy link
Contributor

yguedidi commented Apr 8, 2023

@samsonasik perfect! thank you! I'll make sure to open PRs for the others running all those checks!

samsonasik added a commit that referenced this pull request May 8, 2023
* Revert fix cs (#3579)

This reverts commit e66317a.

* Revert Use isInteger() (#3571)

This reverts commit 59cdfc8.

* Revert Use isBoolean() (#3570)

This reverts commit 437cc5a.

* Revert Use isVoid() (#3574)

This reverts commit 7611dcb.

* Revert Use isString() (#3573)

This reverts commit aa73118.

* Revert Use isFloat() (#3572)

This reverts commit 46301c1.

* Revert Improve ArraySpreadInsteadOfArrayMergeRector (#3568)

This reverts commit 073f2d6.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants