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

Posted to notifications@netbeans.apache.org by GitBox <gi...@apache.org> on 2022/12/22 17:55:33 UTC

[GitHub] [netbeans] lkishalmi commented on pull request #5115: Added more tests BaseUtilities & Pair classes

lkishalmi commented on PR #5115:
URL: https://github.com/apache/netbeans/pull/5115#issuecomment-1363182797

   @lbownik I did not want to really put out on the public mailing list. Most probably the main reason this PR have not got reviewed is the lack of value. These changing old and proven test, while add a few testcases. 
   
   Changing tests without a greater force driving the change (like upgrade due to some version incompatibility) does not add any value.
   
   The added test cases are usually welcome especially when those can be associated with a real issue.


-- 
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: notifications-unsubscribe@netbeans.apache.org

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


---------------------------------------------------------------------
To unsubscribe, e-mail: notifications-unsubscribe@netbeans.apache.org
For additional commands, e-mail: notifications-help@netbeans.apache.org

For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists