fix(bump): only add version files

BREAKING CHANGE: VersionProvider has new method which will break any custom version provider

Author: marcusalstrom

commitizen/commands/bump.py

@@ -344,6 +344,7 @@ def __call__(self) -> None:  # noqa: C901
         )
 
         provider.set_version(str(new_version))
+        files.extend(provider.get_files())
 
         if self.pre_bump_hooks:
             hooks.run(
@@ -420,7 +421,7 @@ def __call__(self) -> None:  # noqa: C901
             out.success("Done!")
 
     def _get_commit_args(self) -> str:
-        commit_args = ["-a"]
+        commit_args = []
         if self.no_verify:
             commit_args.append("--no-verify")
         return " ".join(commit_args)

commitizen/providers/base_provider.py

@@ -34,6 +34,12 @@ def set_version(self, version: str):
         Set the new current version
         """
 
+    @abstractmethod
+    def get_files(self) -> list[str]:
+        """
+        Get the version files
+        """
+
 
 class FileProvider(VersionProvider):
     """
@@ -66,6 +72,9 @@ def set_version(self, version: str):
     def get(self, document: dict[str, Any]) -> str:
         return document["version"]  # type: ignore
 
+    def get_files(self) -> list[str]:
+        return [str(self.file)]
+
     def set(self, document: dict[str, Any], version: str):
         document["version"] = version
 
@@ -87,5 +96,8 @@ def set_version(self, version: str):
     def get(self, document: tomlkit.TOMLDocument) -> str:
         return document["project"]["version"]  # type: ignore
 
+    def get_files(self) -> list[str]:
+        return [str(self.file)]
+
     def set(self, document: tomlkit.TOMLDocument, version: str):
         document["project"]["version"] = version  # type: ignore

commitizen/providers/commitizen_provider.py

@@ -13,3 +13,6 @@ def get_version(self) -> str:
 
     def set_version(self, version: str):
         self.config.set_key("version", version)
+
+    def get_files(self) -> list[str]:
+        return [str(self.config._path)]

commitizen/providers/npm_provider.py

@@ -58,6 +58,15 @@ def set_version(self, version: str) -> None:
                 json.dumps(shrinkwrap_document, indent=self.indent) + "\n"
             )
 
+    def get_files(self) -> list[str]:
+        files = []
+        files.append(str(self.package_file))
+        if self.lock_file.exists():
+            files.append(str(self.lock_file))
+        if self.shrinkwrap_file.exists():
+            files.append(str(self.shrinkwrap_file))
+        return files
+
     def get_package_version(self, document: dict[str, Any]) -> str:
         return document["version"]  # type: ignore
 

commitizen/providers/scm_provider.py

@@ -78,3 +78,7 @@ def get_version(self) -> str:
     def set_version(self, version: str):
         # Not necessary
         pass
+
+    def get_files(self) -> list[str]:
+        # No files for this provider
+        return []