Join GitHub today
GitHub is home to over 28 million developers working together to host and review code, manage projects, and build software together.Sign up
Add time zone to Java 8 datetime mappers #2069
Unless I'm missing something this seems like a bug. I ran into it today while playing with JDBI.
Is there a good reason to keep around the default constructor? It looks like it is only used in tests right now. Those could easily pass an empty optional and get the same result, and that would have made this impossible to miss.