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

php8 support #27

Closed
marcin-derlukiewicz opened this issue Nov 30, 2020 · 8 comments · Fixed by #30
Closed

php8 support #27

marcin-derlukiewicz opened this issue Nov 30, 2020 · 8 comments · Fixed by #30

Comments

@marcin-derlukiewicz
Copy link
Contributor

php8 is released, symfony supports it, could we add support for it?

@NicoHaase
Copy link
Collaborator

Feel free to test it :)

@marcin-derlukiewicz
Copy link
Contributor Author

shouldn't you just add config for travis to test it?

@NicoHaase
Copy link
Collaborator

According to https://travis-ci.com/github/rewieer/TaskSchedulerBundle/builds/205478767, this looks surprisingly good. I'll test it in a productive project to see whether this really was that easy...

@marcin-derlukiewicz
Copy link
Contributor Author

@NicoHaase any news regarding this?

@NicoHaase
Copy link
Collaborator

I'm sorry, but I could not test this in any of my projects yet, as most of them are still waiting for a Doctrine update or other packages to be fully compatible to PHP 8

@madmortigan1
Copy link

Hi,
there is depcreated exception in the following file:
Task/Schedual.php
Line 85.

PHP Deprecated: Required parameter $part follows optional parameter $time in /var/www/html/vendor/rewieer/taskschedulerbundle/Task/Schedule.php on line 85

Adding default null to $part is a quick fix for it

@NicoHaase
Copy link
Collaborator

@madmortigan1 this has already been fixed on master in #26

@NicoHaase
Copy link
Collaborator

@marcin-derlukiewicz thanks for your help getting this release out: this package supports PHP 8 from now on. I'd like to hear from you whether it works as it should, I could not spot any problems with simple tests

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 a pull request may close this issue.

3 participants