Make generated .gitignore more multi-module friendly

Closes gh-927
This commit is contained in:
Andy Wilkinson
2019-06-14 14:31:22 +01:00
parent 2f50cf1c8f
commit 372ff967a1
2 changed files with 10 additions and 7 deletions

View File

@@ -49,8 +49,9 @@ public class GitProjectGenerationConfiguration {
@ConditionalOnBuildSystem(MavenBuildSystem.ID) @ConditionalOnBuildSystem(MavenBuildSystem.ID)
public GitIgnoreCustomizer mavenGitIgnoreCustomizer() { public GitIgnoreCustomizer mavenGitIgnoreCustomizer() {
return (gitIgnore) -> { return (gitIgnore) -> {
gitIgnore.getGeneral().add("/target/", "!.mvn/wrapper/maven-wrapper.jar"); gitIgnore.getGeneral().add("target/", "!.mvn/wrapper/maven-wrapper.jar", "!**/src/main/**",
gitIgnore.getNetBeans().add("/build/"); "!**/src/test/**");
gitIgnore.getNetBeans().add("build/");
}; };
} }
@@ -58,8 +59,9 @@ public class GitProjectGenerationConfiguration {
@ConditionalOnBuildSystem(GradleBuildSystem.ID) @ConditionalOnBuildSystem(GradleBuildSystem.ID)
public GitIgnoreCustomizer gradleGitIgnoreCustomizer() { public GitIgnoreCustomizer gradleGitIgnoreCustomizer() {
return (gitIgnore) -> { return (gitIgnore) -> {
gitIgnore.getGeneral().add(".gradle", "/build/", "!gradle/wrapper/gradle-wrapper.jar"); gitIgnore.getGeneral().add(".gradle", "build/", "!gradle/wrapper/gradle-wrapper.jar", "!**/src/main/**",
gitIgnore.getIntellijIdea().add("/out/"); "!**/src/test/**");
gitIgnore.getIntellijIdea().add("out/");
}; };
} }

View File

@@ -67,8 +67,8 @@ class GitProjectGenerationConfigurationTests {
ProjectDescription description = new ProjectDescription(); ProjectDescription description = new ProjectDescription();
description.setBuildSystem(new GradleBuildSystem()); description.setBuildSystem(new GradleBuildSystem());
description.setPlatformVersion(Version.parse("2.1.0.RELEASE")); description.setPlatformVersion(Version.parse("2.1.0.RELEASE"));
assertThat(generateGitIgnore(description)) assertThat(generateGitIgnore(description)).contains(".gradle", "build/", "!gradle/wrapper/gradle-wrapper.jar",
.contains(".gradle", "/build/", "!gradle/wrapper/gradle-wrapper.jar", "/out/") "out/", "!**/src/main/**", "!**/src/test/**")
.doesNotContain("/target/", "!.mvn/wrapper/maven-wrapper.jar"); .doesNotContain("/target/", "!.mvn/wrapper/maven-wrapper.jar");
} }
@@ -77,7 +77,8 @@ class GitProjectGenerationConfigurationTests {
ProjectDescription description = new ProjectDescription(); ProjectDescription description = new ProjectDescription();
description.setBuildSystem(new MavenBuildSystem()); description.setBuildSystem(new MavenBuildSystem());
description.setPlatformVersion(Version.parse("2.1.0.RELEASE")); description.setPlatformVersion(Version.parse("2.1.0.RELEASE"));
assertThat(generateGitIgnore(description)).contains("/target/", "!.mvn/wrapper/maven-wrapper.jar") assertThat(generateGitIgnore(description))
.contains("target/", "!.mvn/wrapper/maven-wrapper.jar", "!**/src/main/**", "!**/src/test/**")
.doesNotContain(".gradle", "!gradle/wrapper/gradle-wrapper.jar", "/out/"); .doesNotContain(".gradle", "!gradle/wrapper/gradle-wrapper.jar", "/out/");
} }