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

refactor: remove gc_collect_cycles() called twice for PHP < 8.1 #173

Merged
merged 1 commit into from
Aug 2, 2023

Conversation

sad270
Copy link
Contributor

@sad270 sad270 commented Aug 2, 2023

Since #162 we only support PHP 8.1 and PHP 8.2
This code added in #124 is only for PHP 8.0 and PHP 7.4 we don't need it anymore :)

@sad270 sad270 force-pushed the refactor/remove-code-for-php-8.0 branch 2 times, most recently from a814cca to f7850eb Compare August 2, 2023 18:42
Since Roave#162 we only support PHP 8.1 and PHP 8.2
This code added in Roave#124  is only for PHP 8.0 and PHP 7.4
@sad270 sad270 force-pushed the refactor/remove-code-for-php-8.0 branch from f7850eb to 3ceeced Compare August 2, 2023 18:44
Copy link
Member

@Ocramius Ocramius left a comment

Choose a reason for hiding this comment

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

Thanks @sad270!

@Ocramius Ocramius self-assigned this Aug 2, 2023
@Ocramius Ocramius added the enhancement New feature or request label Aug 2, 2023
@Ocramius Ocramius added this to the 1.5.0 milestone Aug 2, 2023
@Ocramius Ocramius merged commit ede929f into Roave:1.5.x Aug 2, 2023
8 checks passed
@sad270 sad270 deleted the refactor/remove-code-for-php-8.0 branch August 2, 2023 18:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants