Skip to content
This repository has been archived by the owner on Aug 20, 2018. It is now read-only.

PHP 7.2 Compatibility #51

Merged
merged 2 commits into from
Oct 14, 2017
Merged

PHP 7.2 Compatibility #51

merged 2 commits into from
Oct 14, 2017

Conversation

konecnyjakub
Copy link
Contributor

PHP 7.2 does not allow to use Object as class name, any attempt to extend it (Nette\Object here) results in a fatal error. So I used trait Nette\SmartObject instead.
Tests on PHP 7.2 still fail because of Kdyby packages which still use Nette\Object. If one does not use Doctrine, there should be no errors (I believe).

@mrtnzlml
Copy link
Member

Hi, please rebase it on top of the master branch (green tests). Also if the tests are failing then I cannot really merge it, sorry.

Please also note this warning in "readme" file:

At this moment I don't have time, energy and money to maintain this project. But it's a shame so if you depend on this project and you want to become a sponsor or develop it further please don't hesitate to contact me. Otherwise, I am not able to guarantee bright future of this repo... :)

Thanks for understanding.

@konecnyjakub
Copy link
Contributor Author

konecnyjakub commented Oct 13, 2017

Rebase done. I can allow failures with 7.2 or do not test on it at all. What would you prefer?

@mrtnzlml
Copy link
Member

Allow it in failures makes sense to me until somebody will fix it...

@konecnyjakub
Copy link
Contributor Author

So allow failures? Do note that these errors do not come from this package's code but its dependencies.

@mrtnzlml
Copy link
Member

Yes, allow failures (until the 3rd party libraries are fixed).

@konecnyjakub
Copy link
Contributor Author

Alright, done.

@mrtnzlml
Copy link
Member

Good, also please keed unified coding standard with this project (use tabs please).

@konecnyjakub
Copy link
Contributor Author

Sorry about that. It is fixed now.

@mrtnzlml mrtnzlml merged commit cfe8471 into mrtnzlml-archive:master Oct 14, 2017
@mrtnzlml
Copy link
Member

Ok, merged. Thanks.

@konecnyjakub konecnyjakub deleted the php7.2 branch October 14, 2017 14:09
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