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

2019-10-24 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-546062013
 
 
   @suztomo 
   Thx for contributing! Let's keep next commits comming more.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-10-24 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-545939445
 
 
   The 
[build](https://builds.apache.org/job/maven-box/job/maven/job/MRESOLVER-93/) 
for integration tests with maven and resolver will be running after Jenkins 
restart.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-10-23 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-545673852
 
 
   The build is running 
[here|https://builds.apache.org/job/maven-box/job/maven-resolver/job/MRESOLVER-93/]


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-09-02 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-527045121
 
 
   Any objections to merge this PR?
   The local build is successful.


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-08-21 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523406136
 
 
   @bentmann 
   Hi, you are the author of `PathRecordingDependencyVisitor`. Can you comment 
this bug fix?


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-08-20 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523154372
 
 
   I expect more developers to join , but we may add one more commit later on. 
So that the line `paths.add( Arrays.asList( path ) );` and use `addAll()`. 


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services


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

2019-08-20 Thread GitBox
Tibor17 commented on issue #39: [MRESOLVER-93] - PathRecordingDependencyVisitor 
to handle 3 cycles
URL: https://github.com/apache/maven-resolver/pull/39#issuecomment-523064813
 
 
   @suztomo 
   In Jira the affect version is 1.4.1. We did not change this class. Was it 
the same issue in previous versions?


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.
 
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org


With regards,
Apache Git Services