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

feat: add support for Composer 2.x #6

Draft
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

owenvoke
Copy link

I've updated this to support Composer 2.x as well as 1.x, it's also now tests on GitHub Actions, but I can revert to Travis CI if that would be preferable.

As PHP 7.3 is the oldest supported PHP version, I have removed testing for earlier versions, so it now tests against PHP 7.3, 7.4, and 8.0 in CI.

The tests do throw warnings as they are using createPartialMock() with methods that don't exist on the base class.

I've only got one project that's using this, so it would need some further testing to confirm it's all working.

@bobey
Copy link
Member

bobey commented Jun 11, 2021

Hi @owenvoke, thanks for this PR.

This project is not actively maintained by our Team as we don't use it anymore. That being said, I can't see any reason to not merge your PR.

Let me know when you consider your PR ready.

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