Potential issue on master - after merging #276
michalbundyra opened this issue · 2 comments
michalbundyra commented
As noted to @ezimuel on Slack we have potential issue here:
bef0d5a#diff-1b78954777bbbf9668cb36aef77507c2R248 in case of else
(after merging PR #276).
It should be verified by unit tests and fixed.
/cc @remicollet
remicollet commented
I don't think we can have something else than an aray or a string.
So I deliberately keep this missing else, but feel free to improve
ezimuel commented
@remicollet this means we can replace elseif (condition)
with a simple else
?