This repository has been archived by the owner on Oct 24, 2024. It is now read-only.

fix unnecessary null checks in SocketClient #2

Merged

fabio-guerreiro merged 1 commit into main from fix_websocket_on_connection_lost

Jul 26, 2024

fabio-guerreiro commented Jul 26, 2024

There are some simple steps to get your PR merged, that are the following:

Describe your PR and why a maintainer should merge it;
Put the same description inside the commit body otherwise if we change github hosting you application will be based on a instable source code;
Write the commit header in the way that it is following these simple rules [1]
Make sure that your PR will pass the CI
Wait for an review 😄 that will not happen if one of the previous step is missing.

Based on the fixes applied here: zino-hofmann#1410


          remove unnecessary null checks in SocketClient

760e407

ThHareau approved these changes

View reviewed changes

fabio-guerreiro merged commit 3c5c994 into main

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet