Closed jdufresne closed 5 years ago
The value is being used as a boolean for branching, so use a boolean rather than an int. Easier to understand for the reader.
Additionally, always use a keyword argument rather than a bare 0/1 or False/True to provide context to the reader.
Coverage remained the same at 97.601% when pulling 695c8f440dbd0f9a0112a37ee33fa95d60b85d68 on jdufresne:o-bool into 3697acd58137bec34ef7dc812f69f3b14990a9bb on Alir3z4:master.
The value is being used as a boolean for branching, so use a boolean rather than an int. Easier to understand for the reader.
Additionally, always use a keyword argument rather than a bare 0/1 or False/True to provide context to the reader.