18. 贡献

Spring Cloud 在非限制性 Apache 2.0 许可证下发布, 并遵循非常标准的 Github 开发流程,使用 Github tracker 和 merge pull requests 到 master 中。如果需要帮助, 即使是微不足道的事情,也请不要犹豫,但是 请遵循以下指南。spring-doc.cn

18.1. 签署贡献者许可协议

在我们接受重要的补丁或拉取请求之前,我们需要您签署贡献者许可协议。 签署贡献者协议不会授予任何人对主 仓库,但这确实意味着我们可以接受您的贡献,并且您将获得一个 如果我们这样做,请注明作者。可能会要求活跃的贡献者加入核心团队,并且 赋予合并拉取请求的能力。spring-doc.cn

18.2. 行为准则

本项目遵循 行为。参与即表示您同意遵守此准则。请举报 不可接受的行为 [email protected]。spring-doc.cn

18.3. 代码约定和内务管理

这些对于拉取请求都不是必需的,但它们都会有所帮助。他们也可以是 在原始拉取请求之后但在合并之前添加。spring-doc.cn

  • 使用 Spring Framework 代码格式约定。如果您使用 Eclipse 你可以使用 Spring 中的文件导入格式化程序设置 Cloud Build 项目。如果使用 IntelliJ,则可以使用 Eclipse 代码格式化程序 Plugin 导入相同的文件。eclipse-code-formatter.xmlspring-doc.cn

  • 确保所有新文件都有一个简单的 Javadoc 类注释,至少有一个标签可以标识您,最好至少有一个关于类是什么的段落 为。.java@authorspring-doc.cn

  • 将 ASF 许可证标头注释添加到所有新文件(从现有文件复制 在项目中).javaspring-doc.cn

  • 将自己添加为您进行大量修改的 .java 个文件(更多 比外观变化)。@authorspring-doc.cn

  • 添加一些 Javadocs,如果更改命名空间,还可以添加一些 XSD doc 元素。spring-doc.cn

  • 一些单元测试也会有很大帮助 — 必须有人来做。spring-doc.cn

  • 如果没有其他人正在使用你的分支,请针对当前的 master 进行变基(或 主项目中的 other target branch)。spring-doc.cn

  • 编写提交消息时,请遵循以下约定: 如果您正在修复现有问题,请在提交末尾添加 消息(其中 XXXX 是问题编号)。Fixes gh-XXXXspring-doc.cn

18.4. Checkstyle

Spring Cloud Build 带有一组 checkstyle 规则。您可以在模块中找到它们。该模块下最值得注意的文件是:spring-cloud-build-toolsspring-doc.cn

spring-cloud-build-tools/
└── src
    ├── checkstyle
    │   └── checkstyle-suppressions.xml (3)
    └── main
        └── resources
            ├── checkstyle-header.txt (2)
            └── checkstyle.xml (1)
1 默认 Checkstyle 规则
2 文件头设置
3 默认隐藏规则

18.4.1. Checkstyle 配置

默认情况下,Checkstyle 规则处于禁用状态。要将 checkstyle 添加到您的项目中,只需定义以下属性和插件。spring-doc.cn

pom.xml
<properties>
<maven-checkstyle-plugin.failsOnError>true</maven-checkstyle-plugin.failsOnError> (1)
        <maven-checkstyle-plugin.failsOnViolation>true
        </maven-checkstyle-plugin.failsOnViolation> (2)
        <maven-checkstyle-plugin.includeTestSourceDirectory>true
        </maven-checkstyle-plugin.includeTestSourceDirectory> (3)
</properties>

<build>
        <plugins>
            <plugin> (4)
                <groupId>io.spring.javaformat</groupId>
                <artifactId>spring-javaformat-maven-plugin</artifactId>
            </plugin>
            <plugin> (5)
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-checkstyle-plugin</artifactId>
            </plugin>
        </plugins>

    <reporting>
        <plugins>
            <plugin> (5)
                <groupId>org.apache.maven.plugins</groupId>
                <artifactId>maven-checkstyle-plugin</artifactId>
            </plugin>
        </plugins>
    </reporting>
</build>
1 出现 Checkstyle 错误时构建失败
2 违反 Checkstyle 时构建失败
3 Checkstyle 还分析测试源
4 添加 Spring Java 格式插件,该插件将重新格式化您的代码以通过大多数 Checkstyle 格式规则
5 将 checkstyle 插件添加到您的构建和报告阶段

如果你需要隐藏一些规则(例如,行长度需要更长),那么你只需在 suppressions 下定义一个文件就足够了。例:${project.root}/src/checkstyle/checkstyle-suppressions.xmlspring-doc.cn

projectRoot/src/checkstyle/checkstyle-suppresions.xml
<?xml version="1.0"?>
<!DOCTYPE suppressions PUBLIC
        "-//Puppy Crawl//DTD Suppressions 1.1//EN"
        "https://www.puppycrawl.com/dtds/suppressions_1_1.dtd">
<suppressions>
    <suppress files=".*ConfigServerApplication\.java" checks="HideUtilityClassConstructor"/>
    <suppress files=".*ConfigClientWatch\.java" checks="LineLengthCheck"/>
</suppressions>

建议将 和 复制到您的项目中。这样,将应用一些默认格式规则。您可以通过运行以下脚本来执行此操作:${spring-cloud-build.rootFolder}/.editorconfig${spring-cloud-build.rootFolder}/.springformatspring-doc.cn

$ curl https://raw.githubusercontent.com/spring-cloud/spring-cloud-build/master/.editorconfig -o .editorconfig
$ touch .springformat

18.5. IDE 设置

18.5.1. Intellij IDEA

为了设置 Intellij,您应该导入我们的编码约定、检查配置文件并设置 checkstyle 插件。 可以在 Spring Cloud Build 项目中找到以下文件。spring-doc.cn

spring-cloud-build-tools/
└── src
    ├── checkstyle
    │   └── checkstyle-suppressions.xml (3)
    └── main
        └── resources
            ├── checkstyle-header.txt (2)
            ├── checkstyle.xml (1)
            └── intellij
                ├── Intellij_Project_Defaults.xml (4)
                └── Intellij_Spring_Boot_Java_Conventions.xml (5)
1 默认 Checkstyle 规则
2 文件头设置
3 默认隐藏规则
4 应用大多数 Checkstyle 规则的 Intellij 的项目默认值
5 应用大多数 Checkstyle 规则的 Intellij 的项目样式约定
代码样式
图 1.代码样式

转到 → → → 。然后单击该部分旁边的图标。在那里,单击值并选择选项。导入文件。FileSettingsEditorCode styleSchemeImport SchemeIntellij IDEA code style XMLspring-cloud-build-tools/src/main/resources/intellij/Intellij_Spring_Boot_Java_Conventions.xmlspring-doc.cn

代码样式
图 2.检查配置文件

转到 → → → 。然后单击该部分旁边的图标。在那里,单击 并导入文件。FileSettingsEditorInspectionsProfileImport Profilespring-cloud-build-tools/src/main/resources/intellij/Intellij_Project_Defaults.xmlspring-doc.cn

Checkstyle

要让 Intellij 与 Checkstyle 一起使用,您必须安装插件。建议同时安装 以自动转换 JUnit 断言CheckstyleAssertions2Assertjspring-doc.cn

Checkstyle

转到 → → → 。然后单击部分中的图标。在那里,您必须定义应该从何处选择 checkstyle 规则。在上图中,我们从克隆的 Spring Cloud Build 存储库中挑选了规则。但是,您可以指向 Spring Cloud Build 的 GitHub 存储库(例如,对于 : )。我们需要提供以下变量:FileSettingsOther settingsCheckstyle+Configuration filecheckstyle.xmlraw.githubusercontent.com/spring-cloud/spring-cloud-build/master/spring-cloud-build-tools/src/main/resources/checkstyle.xmlspring-doc.cn

请记住将 设置为 to,因为我们对生产和测试源应用 checkstyle 规则。Scan ScopeAll sources

18.6. 重复查找器

Spring Cloud Build 带来了 ,它允许在 Java Classpath 上标记重复和冲突的类和资源。basepom:duplicate-finder-maven-pluginspring-doc.cn

18.6.1. 重复的 Finder 配置

默认情况下,重复查找器处于启用状态,并将在 Maven 构建阶段运行,但只有在将 添加到项目的 .verifyduplicate-finder-maven-pluginbuildpom.xmlspring-doc.cn

pom.xml
<build>
    <plugins>
        <plugin>
            <groupId>org.basepom.maven</groupId>
            <artifactId>duplicate-finder-maven-plugin</artifactId>
        </plugin>
    </plugins>
</build>

对于其他属性,我们设置了插件文档中列出的默认值。spring-doc.cn

您可以轻松覆盖它们,但将所选属性的值设置为前缀为 .例如,设置为 to order to skip duplicates check in your build.duplicate-finder-maven-pluginduplicate-finder-maven-plugin.skiptruespring-doc.cn

如果您需要将 or 添加到您的设置中,请确保在项目的插件配置部分添加它们:ignoredClassPatternsignoredResourcePatternsspring-doc.cn

<build>
    <plugins>
        <plugin>
            <groupId>org.basepom.maven</groupId>
            <artifactId>duplicate-finder-maven-plugin</artifactId>
            <configuration>
                <ignoredClassPatterns>
                    <ignoredClassPattern>org.joda.time.base.BaseDateTime</ignoredClassPattern>
                    <ignoredClassPattern>.*module-info</ignoredClassPattern>
                </ignoredClassPatterns>
                <ignoredResourcePatterns>
                    <ignoredResourcePattern>changelog.txt</ignoredResourcePattern>
                </ignoredResourcePatterns>
            </configuration>
        </plugin>
    </plugins>
</build>