[GitHub] [maven-resolver] suztomo commented on issue #39: [MRESOLVER-93] PathRecordingDependencyVisitor to handle 3 cycles

suztomo commented on issue #39: [MRESOLVER-93] PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-546074592 Thank you. This is an automated message from the Ap

[GitHub] [maven-resolver] suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-545489001 @Tibor17 Checking in. Is there anything I can do to proceed to merge this? ---

[GitHub] [maven-resolver] suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523065669 (I picked up the highest version) I just updated to include 1.4.0 and 1.3.3 to mean the problem should be there fo

[GitHub] [maven-resolver] suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles

suztomo commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor to handle 3 cycles URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523063825 @Tibor17 I've added illustration on why visited set does not work as expected in [MRESOLVER-93](https://issues.apa