Minor refactoring of Formatter logic#2998
Minor refactoring of Formatter logic#2998jjatie merged 4 commits intoChartsOrg:masterfrom jjatie:minor-formatter-logic
Conversation
Codecov Report
Continue to review full report at Codecov.
|
| { | ||
| return formatter?.string(from: NSNumber(floatLiteral: value)) ?? "" | ||
| } | ||
| return block?(value, axis) ?? |
There was a problem hiding this comment.
I think if let would be more readable in this case.
| { | ||
| return formatter?.string(from: NSNumber(floatLiteral: value)) ?? "" | ||
| } | ||
| return block?(value, entry, dataSetIndex, viewPortHandler) ?? |
There was a problem hiding this comment.
I think if let would be more readable in this case.
There was a problem hiding this comment.
I flip-flopped on this one a lot. I'll change it now.
|
Changes have been made |
|
@petester42 Any further feedback on this one or can it be merged? |
|
Actually the naming of the commit / PR itself could have been better, as it does not change the logic - it's a refactoring :-) |
|
and I already told him to try squash before merging and our idea for managing commits |
No description provided.