Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Support NumberFormat#multipleOf #80

Merged
merged 2 commits into from Sep 24, 2015
Merged

Conversation

georgewfraser
Copy link
Contributor

@cowtowncoder
Copy link
Member

Sounds good, would be happy to merge this. Since it's an API change, it needs to go in 2.7, and I just upgrade master to that.

One practical thing we need before merge is the Contributor License Agreement (CLA). Unless I have asked and gotten one earlier (apologies if so -- I tried to check archives), it is needed once, and then I can merge all contributions.

The usual way is to download CLA from https://github.com/FasterXML/jackson/blob/master/contributor-agreement.pdf, print it, fill & sign, scan and email to info at fasterxml dot com.
Apologies for this extra step, but corporate users are quite strict in checking projects follow basic guidelines for contributions, and this is the simplest workflow we could think of.

Once again for this and other contributions, looking forward to merging them!

@georgewfraser
Copy link
Contributor Author

CLA sent.

@cowtowncoder cowtowncoder added this to the 2.7.0 milestone Sep 24, 2015
cowtowncoder added a commit that referenced this pull request Sep 24, 2015
Support NumberFormat#multipleOf
@cowtowncoder cowtowncoder merged commit caa2b7b into FasterXML:master Sep 24, 2015
cowtowncoder added a commit that referenced this pull request Sep 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants