garydgregory commented on PR #706:
URL: https://github.com/apache/commons-compress/pull/706#issuecomment-3307331487
@ppkarwasz
I understand the intent, and I like the idea of using the tests as examples.
That said, we still need to test this old code. I'm surprised our Jacoco test
coverage limits didn't break, but it's just a game of averages when you add new
code.
I don't see any point in delaying deprecations. In general, when we
introduce a new API that we know replaces an old way of doing things, we
deprecate the old APIs. It feels easier to understand this way, in with the
new, out with the old. Otherwise, deprecations will appear out of the blue and
it'll be a surprise, regardless of what the release notes said for the previous
versions.
To keep things clean and simple, we've not used non-stock JRE annotations in
Commons IIRC, and I think that's nice.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
