Fixes 127 - ZonedDateTime in map keys can now be serialized with zone IDs

[vetler]

Fixes #127 - ZonedDateTime in map keys can now be serialized with zone IDs

[cowtowncoder]

Tough question on versioning. If we wanted to follow SemVer strictly, I think it should be included in 2.11 as there is behavioral change that might not be considered a bug (or it could be, hence ambiguity).
But 2.10.0 is just out -- if we had, say, 2.10.8, I'd say 2.11.

So: @kupci what do you think? I think I am ok with either 2.10.x or 2.11.

[cowtowncoder]

So: I think patch itself is fine. And the only process thing here is that unless I have already asked and received CLA:

/~https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

that would be needed now (just once; no need to file for more contributions)

Thank you @vetler (Kiitos!) for the contribution; looking forward to merging it once we figure out right branch.

[kupci]

Given this requires the SerializationFeature.WRITE_DATES_WITH_ZONE_ID, which it is now working via @vetler 's fix, 2.10.x seems valid. In other words, this is a bug fix, so fits with

Bug fixes not affecting the API increment the patch version,

[cowtowncoder]

Ok, then we'll go with 2.10.

[vetler]

So: I think patch itself is fine. And the only process thing here is that unless I have already asked and received CLA:

/~https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf

Where do I send it?

[vetler]

Build failure seems unrelated to changes?

[kupci]

Yes, seems unrelated to changes. I wonder if there are still Sonatype issues, as this looks like the same error as previously with the 2.10 release. There's a ticket with Sonatype opened already.

Checksum validation failed, expected 088cd9f174c370442f12200765610c3a98bfb69e but is ae19b6d3ac494b56b7134f261d60b6d7aa9e89d2 -> [Help 1]

[kupci]

And here's where to send the CLA (copying from another PR):

Simplest way is usually to print, fill & sign, scan, email to info at fasterxml dot com.

[cowtowncoder]

Sonatype fixed the problem with jackson-annotations, but it is possible to have a cached copy -- so it may make sense to rm either whole Maven cache, or at least:

rm ~/.m2/repository/com/fasterxml/jackson/core/jackson-annotations/

@vetler yes, send to info at fasterxml dot com

[cowtowncoder]

CLA received, proceed with merging.