michael-o commented on pull request #77: URL: https://github.com/apache/maven-resolver/pull/77#issuecomment-818655591
> > > @michael-o just my 5 cents: "long running PRs" (this one is slowly 1/2 year old 😄 ) suffer from very same issues as "long running feature branches": is easy to become stale, forgotten, fall into oblivion... I don't think we need to "fix" last straw on every line of code here, in this PR (comments are becoming unmanageable huge and long threads). You already did tremendous work on improving original PR (you pointed out places for simplifying, tidy up and removing "non logical" bits), so IMO let's merge this and then we can continue "step by step" (or "baby steps" as one would say) improving overall code base. Solving _everything at once_ will take this linger even more.... This is bascially my fault because it is a huge change and I have very little time... I generally agree, but I need [this question](https://github.com/apache/maven-resolver/pull/77#issuecomment-817367241) answered first to complete my understanding of your code. As soon as this is answered I will perform the tests and am good to merge this week hopefully. -- 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: [email protected]
