-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Merge tests and pom and code #515
Merge tests and pom and code #515
Conversation
Document bigInt and bigDec behavior to determine what can be changed
…ent. (Will currently assterted to an Exception).
…ing_conversion_to_XML Showing issue of illegal node names with possible underscore-replacement. (Will currently assterted to an Exception).
Agreed, it's just a matter of replacing the unit test license with the lib license. This can be done at any time between the commit and the next release. We haven't done alpha releases yet, but this may be a good candidate.
Getting Gradle to work should be trivial, no? No objection going forward to support build mechanisms like Maven as long as it can be done in a way that does not impact existing users.
The lib code is already Java 7 compatible, we just need to backport the unit test code to run in Java7 too. I think (hope) that can be done without significant loss of coverage. |
Regarding the unit tests:
|
If you plan to switch to gradle as a build tool, you'll find a tutorial for the maven central deployment here: |
@BGehrels FYI I reached out to you via email found on your gehrels.info page, regarding Maven Central permissions. |
Confirmed, thanks. |
@BGehrels Hi, I need to hear back from you regarding Maven Central. It's the only item blocking this PR. |
Until now, we have had a quite unhappy situation:
As a result, we have
So: this PR unifies all those three repos into one.
Whats left to do afterwards:
Side note:
For some reason, github shows a lot of files as changed that do not show changes locally (apart from being moved). I don't know why.