You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@accumulo.apache.org by GitBox <gi...@apache.org> on 2018/01/12 04:19:06 UTC
[GitHub] ctubbsii opened a new pull request #357: ACCUMULO-4611 Deprecate commons config in api
ctubbsii opened a new pull request #357: ACCUMULO-4611 Deprecate commons config in api
URL: https://github.com/apache/accumulo/pull/357
In 3 commits:
# Bump to version 1.9.0 as required by semver for API additions, and so we can deprecate and remove the offending API in 2.0.0 as discussed on the mailing list
# Update apilyzer rules to find offending APIs with respect to commons-config
# Fix
The third "fix" commit is the most interesting one. If necessary, I can push the first two to a branch, and then then do a pull request with just the last commit against that branch, so it's easier to review on its own. Reviewers, let me know if you want me to do that.
@joshelser Tagging you here for review, b/c GitHub won't show your GitHub user name in the "Request reviewers" dialog. Probably b/c you haven't sync'd with GitBox to be added to the repo? Not sure.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
With regards,
Apache Git Services