Skip to content

[TASK] rector: add php70 rule set #549

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

Merged
merged 2 commits into from
Jun 17, 2024

Conversation

ziegenberg
Copy link
Contributor

@ziegenberg ziegenberg commented Jun 17, 2024

This enables the rector rule set for PHP 7.0.

@oliverklee
Copy link
Collaborator

Hi, thanks! Please let‘s have separate PRs for the cleanup and the added ruleset (with the cleanup PR first).

@ziegenberg
Copy link
Contributor Author

Here is the cleanup PR to be merged first: #552

@ziegenberg ziegenberg force-pushed the add-rector-ruleset-php70 branch from 7b04d63 to 7c331f5 Compare June 17, 2024 18:56
@ziegenberg
Copy link
Contributor Author

I dropped the commit with the cleanups from this PR. Sorry for the mess 🙈

@oliverklee
Copy link
Collaborator

Thanks! And there’s no need to apologize.

I’ve merged the cleanup PR now. So this PR here is ready for a rebase now.

Signed-off-by: Daniel Ziegenberg <daniel@ziegenberg.at>
@ziegenberg ziegenberg force-pushed the add-rector-ruleset-php70 branch from 7c331f5 to 76aee91 Compare June 17, 2024 19:15
@ziegenberg
Copy link
Contributor Author

I rebased.

Copy link
Collaborator

@oliverklee oliverklee left a comment

Choose a reason for hiding this comment

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

Thanks!

@oliverklee oliverklee merged commit 74edbec into MyIntervals:main Jun 17, 2024
18 checks passed
@ziegenberg ziegenberg deleted the add-rector-ruleset-php70 branch June 17, 2024 20:09
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.

2 participants