[GitHub] [maven-surefire] SimonBaars commented on a diff in pull request #573: [SUREFIRE-2122] Document workaround for maven-shade-plugin in IntelliJ

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

SimonBaars commented on code in PR #573:
URL: https://github.com/apache/maven-surefire/pull/573#discussion_r1007962774


##########
README.md:
##########
@@ -81,7 +81,10 @@ Build the Surefire project using **Maven 3.2.5+** and **JDK 1.8+**.
      * Check module `surefire-grouper` has source folder `target/generated-sources/javacc`.
        If not, add it manually in the module's project properties
 
-* Setup for development in [IntelliJ IDEA](https://www.jetbrains.com/idea/) should work out of the box.
+* Setup for development in [IntelliJ IDEA](https://www.jetbrains.com/idea/) works largely out of the box. There may be unresolved symbols because [IntelliJ does not support the maven-shade-plugin](https://youtrack.jetbrains.com/issue/IDEA-126596). As a workaround:
+
+    * In IntelliJ, right click `surefire-shared-utils/pom.xml` and choose "Maven" -> "Ignore Projects". Don't remove the module if prompted.
+    * Right click the parent `pom.xml` and choose "Maven" -> "Reload project"

Review Comment:
   Hey, thanks for the feedback! I checked, and indeed your instructions also work. W.r.t. this:
   > * Setup for development in [IntelliJ IDEA](https://www.jetbrains.com/idea/) should work out of the box.
   
   I propose changing it to reflect these setup steps. I'll update my PR.



-- 
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: issues-unsubscribe@maven.apache.org

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