https://github.com/yetanalytics/flint/pull/27

(If only there was an easy way to add people from outside one’s GitHub org as reviewers; otherwise I’d add you as a reviewer directly.)

OMG!!! This looks fantastic! Thanks 🙇 I’ve skimmed over the PR briefly and made some small comments, but I’ve not had time to look in more depth.

Thanks for taking a look! I didn’t see any comments that you left though?

ahh sorry I’d added them one by one in a review; and forgot to submit it.

As I said though these are just from skimming over the diffs; and I’d like to find some more time to dig into it. Though It’ll likely be next week before I get to look at it properly.

generally though I think this looks great! 🙇

I responded to the comments that you left. The only one I disagree with was the one about java.util.Date for reasons I explain in my followup, but otherwise I took into account your suggestions.

Just added support for OffsetDateTime, ZonedDateTime, LocalDateTime, and their date- and time-only equivalents