You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@maven.apache.org by GitBox <gi...@apache.org> on 2023/01/15 19:25:35 UTC

[GitHub] [maven-integration-testing] michael-o commented on pull request #233: [MNG-7661] Replace deprecated 'verifier.newDefaultFilterProperties()'

michael-o commented on PR #233:
URL: https://github.com/apache/maven-integration-testing/pull/233#issuecomment-1383231148

   > @slawekjaranowski, @michael-o thanks for reviewing and merging this!
   > 
   > I know this big PRs are hard to review (lots of files, Github UI not even rendering everything, etc). My thinking was that removing deprecated calls is good way to get familiar with the code base also do something somewhat useful in the process.
   > 
   > That being said, I recently learned (https://twitter.com/forked_franz/status/1614191087517679624) that these kind of PRs may not be the best way to start contributing, so I will try to refrain from these big bang changes (at least until I earn some more credit from other smaller PRs).
   
   That tweets sounds like bullshit to me. The change you have done was big, yes, but simple and necessary anyway.


-- 
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.

To unsubscribe, e-mail: issues-unsubscribe@maven.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org