fix off-by-one bug for hamster export tsv#577
fix off-by-one bug for hamster export tsv#577GeraldJansen wants to merge 2 commits intoprojecthamster:masterfrom GeraldJansen:off-by-one
Conversation
|
@elbenfreund Hmm, I signed the CLA long ago. Now if I click on the CLA banner I get this message: Error |
|
My appologies @GeraldJansen as explained here I triggered a project wide refresh when I started working on |
|
Okay, the comment is gone, but I still see "Some checks haven't completed yet" and "license/cla pending". |
|
Yes. As explained in this comment this will probably change once another push is triggered. Anyway, it is not a mandatory check, so it should not impose on anyone's work. |
|
Looking good :) |
|
Good, there must never be a CLA on this repo. I was curious about the origin of this bug, see #580. |
|
Closed in favour of better #580. Yay! |
This PR is a workaround for a bug concerning the range.end in some date ranges (#576).
A better fix would address the issue in hamster.lib.datetime.Range.from_start_end(), but my attempts there created other problems.