cstamas commented on pull request #77:
URL: https://github.com/apache/maven-resolver/pull/77#issuecomment-818646120


   @michael-o just my 5 cents: "long running PRs" (this one is slowly 1/2 year 
old :smile: ) 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 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]


Reply via email to