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.

Sign up for GitHub

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

Jump to bottom

Fix operator precedence in connection factory #1405

Merged

tillkruss merged 1 commit into predis:v2.x from TobiasBengtsson:patch-1

Sep 26, 2023

Contributor

TobiasBengtsson commented Sep 23, 2023

&& binds tighter than ?? so a parentheses is needed for desired result


          Fix operator precedence

51cddb0

&& binds tighter than ?? so a parentheses is needed for desired result

TobiasBengtsson requested a review from tillkruss as a code owner

September 23, 2023 20:31

coveralls commented Sep 23, 2023

coverage: 80.208%. remained the same when pulling 51cddb0 on TobiasBengtsson:patch-1 into 577bd39 on predis:v2.x.

tillkruss approved these changes

View reviewed changes

tillkruss self-assigned this

tillkruss added the bug label

tillkruss merged commit 5c8b19e into predis:v2.x

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

Labels

bug