[GitHub] [maven-surefire] qerub edited a comment on pull request #262: Refer to correct property in skipping-tests doc

[GitBox <gi...@apache.org>]

qerub edited a comment on pull request #262:
URL: https://github.com/apache/maven-surefire/pull/262#issuecomment-581180096


   > The author of the doc separated the text been dedicated to the Surefire and Failsafe plugin using IF-ELSE decision making in one doc file. That's the (1). We do not copy the doc from Surefire module to the Failsafe module and we only switch few lines of text which is maybe 10%.
   
   The docs are separated using IF-ELSE but the author of the current documentation clearly wanted to document both `skipITs` (firstly) **and** `skipTests`. This is the section in its entirety:
   
   ```patch
     To skip running the tests for a particular project, set the <<skipITs>> property to <<true>>.
    
    +---+
    […long example with <skipITs>true</skipITs>…]
    +---+
    
     You can also skip the tests via the command line by executing the following command:
    
    +---+
   -mvn install -DskipITs
   +mvn install -DskipTests
    +---+
    
     Since <<<skipTests>>> is also supported by the ${thatPlugin} Plugin, this will have the effect
     of not running any tests.  If, instead, you want to skip only the integration tests
     being run by the ${thisPlugin} Plugin, you would use the <<<skipITs>>> property instead:
    
    +---+
    mvn install -DskipITs
    +---+
   ```
   
   Please either merge or close this PR instead of leaving it hanging.


----------------------------------------------------------------
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:
users@infra.apache.org