junichi11 commented on a change in pull request #2208:
URL: https://github.com/apache/netbeans/pull/2208#discussion_r445199155
##########
File path:
php/php.latte/test/unit/src/org/netbeans/modules/php/latte/parser/LatteParserErrorTest.java
##########
@@ -43,18 +43,6 @@ public void testIssue245728_03() throws Exception {
checkErrors("testfiles/parser/issue245728_03.latte");
}
- public void testIssue245728_04() throws Exception {
Review comment:
I don't think that it's good to remove them completely. They should have
been added for some reason. Someone might be able to fix them in the future
(although I didn't check what they were tests for, I'm not sure whether we can
fix). So, we should comment them out with the reason(e.g. test files were not
donated) if you don't fix them.
----------------------------------------------------------------
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]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
For further information about the NetBeans mailing lists, visit:
https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
