diff options
| author | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-03-20 18:37:22 +0700 |
|---|---|---|
| committer | Alexander Kriegisch <Alexander@Kriegisch.name> | 2021-03-21 02:01:18 +0700 |
| commit | a1867b05ba6443d32abc4049c26b92fc226d6f78 (patch) | |
| tree | 451e969e0aa7c5998d84cdce0d43f3f1a32d1461 /pom.xml | |
| parent | fa8369ebb82cd9668943173b4f04868b55ab4260 (diff) | |
| download | aspectj-a1867b05ba6443d32abc4049c26b92fc226d6f78.tar.gz aspectj-a1867b05ba6443d32abc4049c26b92fc226d6f78.zip | |
Remove module 'run-all-junit-tests' from root POM -> speed up the build
This module must be a relic from a test runner module once existing
during the Ant build era, but transferred and kept alive in the Maven
build. Actually, it almost doubles build time by running virtually all
tests in all modules again when doing 'mvn test' from the project root.
For now I only removed the module from the root POM, leaving behind
comments there, in the module POM and in the now @Deprecated class
RunTheseBeforeYouCommitTests.
Signed-off-by: Alexander Kriegisch <Alexander@Kriegisch.name>
Diffstat (limited to 'pom.xml')
| -rw-r--r-- | pom.xml | 8 |
1 files changed, 7 insertions, 1 deletions
@@ -37,9 +37,15 @@ <module>testing-drivers</module> <module>tests</module> <module>build</module> - <module>run-all-junit-tests</module> <module>docs</module> + <!-- + These are redundant, making almost all tests run 2x. You can still open and run the module manually for now. + But why not just run 'mvn test' on top level? + TODO: After feedback from Andy Clement, probably just remove the whole module. + --> + <!--<module>run-all-junit-tests</module>--> + <!-- create the important artifacts we care about --> <module>aspectjrt</module> <module>aspectjweaver</module> |