fix: support both snake_case and camelCase versions of impression data #800
Triggered via pull request
September 25, 2024 15:54
gastonfournier
Status
Failure
Total duration
1m 15s
Artifacts
–
tests.yaml
on: pull_request
Matrix: Test code style
Matrix: Static analysis
Matrix: Tests
Matrix: Tests (transpiled versions)
Annotations
4 errors
|
Test code style (8.3)
Process completed with exit code 8.
|
|
Static analysis (8.3):
src/Repository/DefaultUnleashProxyRepository.php#L44
Parameter #1 $response of class Unleash\Client\DTO\DefaultProxyFeature constructor expects array{name: string, enabled: bool, variant: array{name: string, enabled: bool, payload?: array{type: string, value: string}}, impression_data: bool, impressionData: bool}, array{name: string, enabled: bool, variant: array{name: string, enabled: bool, payload?: array{type: string, value: string}}, impressionData: bool} given.
|
|
Static analysis (8.3):
src/Repository/DefaultUnleashProxyRepository.php#L79
Parameter #1 $response of class Unleash\Client\DTO\DefaultProxyFeature constructor expects array{name: string, enabled: bool, variant: array{name: string, enabled: bool, payload?: array{type: string, value: string}}, impression_data: bool, impressionData: bool}, array{name: string, enabled: bool, variant: array{name: string, enabled: bool, payload?: array{type: string, value: string}}, impressionData: bool} given.
|
|
Static analysis (8.3)
Process completed with exit code 1.
|