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

Add bundled signature for Apache Commons IO 2.6 #133

Merged
merged 1 commit into from
Oct 24, 2017

Conversation

joschi
Copy link
Contributor

@joschi joschi commented Oct 24, 2017

This PR adds bundled signatures for the recently released Apache Commons IO 2.6.

It's a bit lazy because it simply includes the previous bundled signatures, though. 😉

@uschindler
Copy link
Member

Thanks @joschi - when I released latest version, 2.6 was not yet out. But I aggree, this would be a first step. The last time I was looking into the API differences between previous version and new version, so I was able to quickly figure out new "forbidden" apis. It would be good to do this before next relaese.

Otherwise looks fine (similar to the other signature files).

@uschindler uschindler added this to the 2.5 milestone Oct 24, 2017
@uschindler uschindler self-assigned this Oct 24, 2017
@uschindler uschindler merged commit 196042e into policeman-tools:master Oct 24, 2017
@uschindler
Copy link
Member

Thanks!

@joschi joschi deleted the commons-io-2.6 branch October 25, 2017 06:43
@uschindler
Copy link
Member

Hi @joschi,
I checked the differences 2.5->2.6 and I found no new charset related issues:
https://abi-laboratory.pro/java/tracker/compat_report/commons-io/2.5/2.6/f223d/bin_compat_report.html

@joschi
Copy link
Contributor Author

joschi commented Mar 28, 2018

@uschindler Thanks for the feedback!

And of course thanks for continuously maintaining this fine project. 😃

@uschindler
Copy link
Member

Release including Java 10 support is coming soon. It's in staging and it's currently tested with Lucene, Solr, and Elasticsearch!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

2 participants