hboutemy commented on a change in pull request #10:
URL: https://github.com/apache/maven-ear-plugin/pull/10#discussion_r484577279



##########
File path: src/it/skinny-wars/verify.bsh
##########
@@ -35,12 +39,44 @@ String[] includedEntries = {
     "WEB-INF/web.xml",
     "META-INF/MANIFEST.MF"
 };
+
+// Refer to org.apache.maven.archiver.MavenArchiver.parseOutputTimestamp

Review comment:
       my idea to simplify the code is not to calculate expected Date, but 
check that everything has the same date
   
   I'll update the IT accordingly: thanks for the great work, you did the 
important part that I didn't know (I'm not a maven-ear-plugin user myself), I 
can finish the little details




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