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

Prophecy.next and PHP version support #445

Closed
ciaranmcnulty opened this issue Oct 3, 2019 · 4 comments
Closed

Prophecy.next and PHP version support #445

ciaranmcnulty opened this issue Oct 3, 2019 · 4 comments

Comments

@ciaranmcnulty
Copy link
Member

ciaranmcnulty commented Oct 3, 2019

We've never had a second major version, and there are some issues:

I propose we adopt a public released date (probably in 2020) for Prophecy 2.0, with a higher version of PHP support at minimum and create a branch for it now. We can then target PRs that remove legacy code and introduce BC breaks to that branch.

We'd need to also make it clear what the support policy for Prophecy 1.0 was

I'd be happy to 'release manage' it.

@stof @everzet what do you think?

@sebastianbergmann Do you have any input here? PHPUnit is one of Prophecy's largest usage drivers (probably more than PhpSpec)

@stof
Copy link
Member

stof commented Oct 3, 2019

(for instance, using anonymous classes for doubling)

this is a bad idea IMO. My own project has a memory issue related to Prophecy due to the number of mock class created (which can never be freed from memory due to the way PHP works). What I would like to work on (but never had the time for that) would be to force always using a cached doubler instead (to avoid create thousands of similar double classes during the testsuite run). Using anonymous classes would be opposite to that.

@stof
Copy link
Member

stof commented Oct 3, 2019

Note that dropping support for dead PHP versions might even be done in a minor version, assuming that tools based on Prophecy don't depend on that support. PHPUnit maintained versions have already dropped them. What is the min PHP requirement of maintained PhpSpec versions @ciaranmcnulty ?

@ciaranmcnulty
Copy link
Member Author

@stof Phpspec doesn't support <7.1 currently

I find version bumps during majors easier to handle but we can discuss

(mock class thing was an example, we'd need to profile that sort of change anyhow right)

@ciaranmcnulty
Copy link
Member Author

Closing this, there wasn't any meaningful input on whether we should do it or not.

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

No branches or pull requests

2 participants