We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Sometimes, it makes sense and the code is more readable as an if/else instead of a ternary operator.
One such cases is when there's an execution flow that happens to return a value:
if (this.Url.IsLocalUrl(returnUrl)) { return this.Redirect(returnUrl); } else { return this.RedirectToAction("Index"); }
I want to see that it's following different branches and not just return one of two possible values.
The text was updated successfully, but these errors were encountered:
Makes sense to me.
Sorry, something went wrong.
I will also be changing CC0014 as the argument is the same there.
CC0014
ae594ee
Change TernaryOperatorAnalyzer to use Info instead of Warning
0d818d0
Closes code-cracker#520
giggio
No branches or pull requests
Sometimes, it makes sense and the code is more readable as an if/else instead of a ternary operator.
One such cases is when there's an execution flow that happens to return a value:
I want to see that it's following different branches and not just return one of two possible values.
The text was updated successfully, but these errors were encountered: