feat: Add git-commit-version flag to control commit creation

invertase · Jan 11, 2024 · 9479e40 · 9479e40
1 parent 123f34b
commit 9479e40
Show file tree

Hide file tree

Showing 2 changed files with 28 additions and 15 deletions.
diff --git a/packages/melos/lib/src/command_runner/version.dart b/packages/melos/lib/src/command_runner/version.dart
@@ -84,10 +84,15 @@ class VersionCommand extends MelosCommand {
       'git-tag-version',
       abbr: 't',
       defaultsTo: true,
-      help:
-          'By default, melos version will commit changes to pubspec.yaml files '
-          'and tag the release. Pass --no-git-tag-version to disable the '
-          'behaviour.',
+      help: 'By default, melos version will tag the release. Pass '
+          '--no-git-tag-version to disable the behaviour.',
+    );
+    argParser.addFlag(
+      'git-commit-version',
+      abbr: 'C',
+      defaultsTo: true,
+      help: 'By default, melos version will commit changes to pubspec.yaml '
+          'files. Pass --no-git-commit-version to disable the behaviour.',
     );
     argParser.addFlag(
       'release-url',
@@ -167,6 +172,7 @@ class VersionCommand extends MelosCommand {
     final updateDependentsConstraints =
         argResults!['dependent-constraints'] as bool;
     final tag = argResults!['git-tag-version'] as bool;
+    final commit = argResults!['git-commit-version'] as bool;
     final releaseUrl = argResults!.optional('release-url') as bool?;
     final changelog = argResults!['changelog'] as bool;
     final commitMessage =
@@ -196,6 +202,7 @@ class VersionCommand extends MelosCommand {
         manualVersions: {packageName: versionChange},
         force: force,
         gitTag: tag,
+        gitCommit: commit,
         releaseUrl: releaseUrl,
         updateChangelog: changelog,
         updateDependentsConstraints: updateDependentsConstraints,
@@ -238,6 +245,7 @@ class VersionCommand extends MelosCommand {
         packageFilters: parsePackageFilters(config.path),
         force: force,
         gitTag: tag,
+        gitCommit: commit,
         releaseUrl: releaseUrl,
         updateChangelog: changelog,
         updateDependentsConstraints: updateDependentsConstraints,

diff --git a/packages/melos/lib/src/commands/version.dart b/packages/melos/lib/src/commands/version.dart
@@ -12,6 +12,7 @@ mixin _VersionMixin on _RunMixin {
     bool updateDependentsConstraints = true,
     bool updateDependentsVersions = true,
     bool gitTag = true,
+    bool gitCommit = true,
     bool? releaseUrl,
     String? message,
     bool force = false,
@@ -59,6 +60,7 @@ mixin _VersionMixin on _RunMixin {
         updateDependentsConstraints: updateDependentsConstraints,
         updateDependentsVersions: updateDependentsVersions,
         gitTag: gitTag,
+        gitCommit: gitCommit,
         releaseUrl: releaseUrl,
         message: message,
         force: force,
@@ -81,6 +83,7 @@ mixin _VersionMixin on _RunMixin {
     bool updateDependentsConstraints = true,
     bool updateDependentsVersions = true,
     bool gitTag = true,
+    bool gitCommit = true,
     bool? releaseUrl,
     String? message,
     bool force = false,
@@ -354,17 +357,19 @@ mixin _VersionMixin on _RunMixin {
       logger.newLine();
     }
 
-    await _gitStageChanges(
-      workspace,
-      pendingPackageUpdates,
-      updateDependentsVersions: updateDependentsVersions,
-    );
-    await _gitCommitChanges(
-      workspace,
-      pendingPackageUpdates,
-      commitMessageTemplate,
-      updateDependentsVersions: updateDependentsVersions,
-    );
+    if (gitCommit) {
+      await _gitStageChanges(
+        workspace,
+        pendingPackageUpdates,
+        updateDependentsVersions: updateDependentsVersions,
+      );
+      await _gitCommitChanges(
+        workspace,
+        pendingPackageUpdates,
+        commitMessageTemplate,
+        updateDependentsVersions: updateDependentsVersions,
+      );
+    }
 
     if (gitTag) {
       await _gitTagChanges(