feat(protobuf): add pre-commit hook for checking backwards compatibil…

…ity for protobuf files
jhipster · Sep 13, 2024 · a3d0f57 · a3d0f57
1 parent a9f0d43
commit a3d0f57
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 1 deletion.
diff --git a/...a/tech/jhipster/lite/generator/server/javatool/protobuf/domain/ProtobufModuleFactory.java b/...a/tech/jhipster/lite/generator/server/javatool/protobuf/domain/ProtobufModuleFactory.java
@@ -110,6 +110,7 @@ public JHipsterModule buildProtobufBackwardsCompatibilityCheckModule(JHipsterMod
         .pluginManagement(protoBackwardsCompatibilityMavenPluginManagement())
         .plugin(protoBackwardsCompatibilityMavenPluginBuilder().build())
         .and()
+      .preCommitActions(stagedFilesFilter("*.proto"), preCommitCommands("() => ['mvn proto-backwards-compatibility:backwards-compatibility-check', 'git add *proto.lock']"))
       .build();
     //@formatter:on
   }

diff --git a/...ch/jhipster/lite/generator/server/javatool/protobuf/domain/ProtobufModuleFactoryTest.java b/...ch/jhipster/lite/generator/server/javatool/protobuf/domain/ProtobufModuleFactoryTest.java
@@ -142,7 +142,17 @@ void shouldBuildModuleForMaven() {
 
       JHipsterModule module = factory.buildProtobufBackwardsCompatibilityCheckModule(properties);
 
-      assertThatModuleWithFiles(module, pomFile())
+      assertThatModuleWithFiles(module, pomFile(), lintStagedConfigFile())
+        .hasFile(".lintstagedrc.cjs")
+        .containing(
+          """
+          module.exports = {
+            '*.proto': () => ['mvn proto-backwards-compatibility:backwards-compatibility-check', 'git add *proto.lock'],
+            '*.{md,json,yml,html,css,scss,java,xml,feature}': ['prettier --write'],
+          };
+          """
+        )
+        .and()
         .hasFile("pom.xml")
         .containing(
           """