v0.1.11 introduces BC-break

[tixastronauta]

Release v.0.1.11 should be a major release as it introduces a bc-break.

My previously working code, now crashes because I'm not injecting a Client into OneSignal constructor. In v.0.1.10 there was no need to inject a Client because a new Guzzle Client was instantiated.

[norkunas]

@tixastronauta constructor in 0.1 doesn't changed and it instantiates guzzle as before you don't inject it

[tixastronauta]

@norkunas from my understanding that is not the case.

Constructor at 0.1.10

public function __construct(Config $config = null, Client $client = null)
{
    $this->config = ($config ?: new Config());
    $this->client = ($client ?: new Client([
        'defaults' => [
            'headers' => [
                'Accept' => 'application/json',
            ],
        ],
    ]));
}

Constructor at 0.1.11

public function __construct(Config $config = null, Client $client = null)
{
    $this->config = ($config ?: new Config());
    if (null !== $client) {
        $this->client = $client;
    }
}

[norkunas]

@tixastronauta sorry, master was target in release. should be good now.

[tixastronauta]

Awesome @norkunas 👍 ! Thank you!