[GitHub] ant-ivy issue #53: suppressed warnings

2017-08-31 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/53 Thanks! This is now merged to upstream. --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-27 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/53 With all due respect, there are some refactorings worth attention IMHO. Besides, some deprecations need to be followed through 😉 --- If your project is set up for it, you can reply to this email

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-27 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/53 Commit https://github.com/apache/ant-ivy/pull/53/commits/414651c8841a09a9d9d4b0365ee8e7227fba8429 in this PR has now been merged to upstream

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-26 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/53 D'uh... getExtends() must respect insertion order -- array straitjacket 😄 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-26 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/53 This is strange, all tests still pass for me locally -- could someone elucidate what the problem is, please? --- If your project is set up for it, you can reply to this email and have your reply

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-25 Thread twogee
Github user twogee commented on the issue: https://github.com/apache/ant-ivy/pull/53 Sorry for additions to PR. I figured out a way to remove the last inconsistencies with generics without breaking the API. This PR will not change any more. I rebased my commits; you're welcome to

[GitHub] ant-ivy issue #53: Suppressed warnings

2017-07-25 Thread jaikiran
Github user jaikiran commented on the issue: https://github.com/apache/ant-ivy/pull/53 Hi Gintas, Given the amount of commits and the changes involved in this PR, it's a bit difficult to review it. I think this PR even includes some of the changes that have already been