Skip to content
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

Logical operators should include parens. #202

Merged
merged 1 commit into from
Oct 12, 2023
Merged

Logical operators should include parens. #202

merged 1 commit into from
Oct 12, 2023

Conversation

jheer
Copy link
Member

@jheer jheer commented Oct 11, 2023

  • Add parens around generated logical (AND, OR) operators when given two or more clause arguments.

This PR should serve as a fix / workaround for #191. Closes #191.

@jheer jheer merged commit f4ab517 into main Oct 12, 2023
2 checks passed
@jheer jheer deleted the jh/logical-parens branch October 12, 2023 20:43
@domoritz
Copy link
Member

Could you make a release with this or document how to make a release so I can make one?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants