TernaryCheck_issue#167 #198

Closed
wants to merge 1 commit into
from

Projects

None yet

3 participants

@alexkravin
Contributor

Now Check became more general, it has 3 options, 2 of them make Check less strict, so user can set it according to his own needs.
Check analyzes the expression for more, than user-defined ternary operators (maxTernaryPerExpressionCountValue)
User can choose options to ignore ternary sequence with explicitly set priority level (expression in braces) and also make Check ignore ternary operators which are isolated in one line.
More detailed description in docs.

@coveralls

Coverage Status

Coverage increased (+0.07%) when pulling 1bd7cb4 on alexkravin:TernaryCheck_issue#167 into b54c734 on sevntu-checkstyle:master.

@coveralls

Coverage Status

Coverage increased (+0.07%) when pulling 8674e02 on alexkravin:TernaryCheck_issue#167 into b54c734 on sevntu-checkstyle:master.

@daniilyar
Member

Merged into main repo. Thank you for contribution!

@daniilyar daniilyar closed this Jul 22, 2014
@alexkravin alexkravin deleted the alexkravin:TernaryCheck_issue#167 branch Sep 5, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment