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

Php vcr master #3

Open
wants to merge 274 commits into
base: master
Choose a base branch
from
Open

Php vcr master #3

wants to merge 274 commits into from

Conversation

Tjeerd
Copy link
Member

@Tjeerd Tjeerd commented Mar 1, 2023

Context

What has been done

How to test

Todo

  • [ ]
  • [ ]

Notes

moufmouf and others added 30 commits October 29, 2018 09:42
Tweaked some PHPDoc statements.
Added some ignore in PHPStan where assertions are already ensuring safety.
Increasing minimum version to have support for Assertion::isResource.
It was not migrated to PHPUnit 7 yet because it was merged after the migration.
AbstractCodeTransform was actually misbehaving if several instances of the same XxxCodeTransform were to be created.
Indeed, a wrapper can be registered, but not unregistered. If 2 instances of CurlCodeTransform were to be created, the registration of the wrapper would be attempted twice. The second one would fail and try again but the wrapper is already registered. This PR registers only if the stream is indeed not already registered.
Was typehinted on the AbstractStorage class instead of the Storage interface.
Added SoapOperation request matcher + unit tests
peter279k and others added 30 commits September 5, 2023 14:04
Add SOAP extension requirement for running tests
…phpcsfixer

Fix pipeline (phpstan & phpcsfixer)
…ment-workspace-docker-image

FEATURE: Added php 8.3 workspace docker image
Remove trailing semicolon in Content-Type header
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.