-
Notifications
You must be signed in to change notification settings - Fork 2
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
Support symfony 7 #49
Conversation
0fb85a1
to
72d1292
Compare
phpunit.xml.dist
Outdated
@@ -1,29 +1,20 @@ | |||
<?xml version="1.0" encoding="UTF-8"?> | |||
<phpunit xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" | |||
xsi:noNamespaceSchemaLocation="http://schema.phpunit.de/6.1/phpunit.xsd" | |||
xsi:noNamespaceSchemaLocation="https://schema.phpunit.de/10.5/phpunit.xsd" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
use vendor/phpunit/phpunit/phpunit.xsd
instead
public function __construct(string $message, Request $request) | ||
public function __construct( | ||
string $message, | ||
public readonly Request $request) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
format
string $message, | ||
public readonly ResponseInterface $response, | ||
public readonly Request $request, | ||
?\Throwable $previous = null) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
format
public function resetHeaders(): void | ||
{ | ||
$this->headers = []; | ||
} | ||
|
||
/** | ||
* @deprecated use getAuthorizationHeaders method instead | ||
*/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
write about it on release note
private $objects; | ||
|
||
public function __construct(array $objects) | ||
public function __construct(public array $objects) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ContentRemoval
@param
* @param bool|int|string $value | ||
*/ | ||
public function addMustFilter(string $key, $value): self | ||
public function addMustFilter(string $key, bool|int|string $value): self | ||
{ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
below conditions can be removed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Then it will throw different exception, is it ok?
* @dataProvider \Answear\LuigisBoxBundle\Tests\DataProvider\RequestDataProvider::forContentUpdate() | ||
*/ | ||
#[Test] | ||
#[DataProvider('forContentUpdate')] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why not from RequestDataProvider
? It is impossible to load from external static method?
AC