lukaszlenart commented on code in PR #730:
URL: https://github.com/apache/struts/pull/730#discussion_r1296942247


##########
.github/workflows/maven.yml:
##########
@@ -49,10 +49,9 @@ jobs:
           distribution: adopt
           java-version: ${{ matrix.java }}
       - name: Build with Maven on Java ${{ matrix.java }}
-        if: matrix.java != '11'
         run: mvn -B -V -DskipAssembly test --no-transfer-progress

Review Comment:
   This can lead to failed tests as quite often builds of different PRs are run 
on the same build agent and Jetty can not bind to port 8090 which already used. 
I will keep `test` phase for the JDK 8 & 11.



-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to