Merge pull request #123 from edgardmessias/improved_changelists

Added options to ignore changelist on commit (#99 and #120)

Author: JohnstonCode

package.json

@@ -256,10 +256,14 @@
         },
         "svn.multipleFolders.ignore": {
           "type": "array",
-          "minimum": 4,
           "description": "Folders to ignore using SVN",
           "default": ["**/.git", "**/.hg", "**/vendor", "**/node_modules"]
         },
+        "svn.sourceControl.ignoreOnCommit": {
+          "type": "array",
+          "description": "Changelists to ignore on commit",
+          "default": ["ignore-on-commit"]
+        },
         "svn.sourceControl.showExternal": {
           "type": "boolean",
           "description":

src/commands.ts

@@ -177,8 +177,17 @@ export class SvnCommands {
       picks.push(new ChangeListItem(repository.changes));
     }
 
+    const svnConfig = workspace.getConfiguration("svn", Uri.file(repository.workspaceRoot));
+    const ignoreOnCommitList = svnConfig.get<string[]>(
+      "sourceControl.ignoreOnCommit",
+      []
+    );
+
     repository.changelists.forEach((group, changelist) => {
-      if (group.resourceStates.length) {
+      if (
+        group.resourceStates.length &&
+        !ignoreOnCommitList.includes(changelist)
+      ) {
         picks.push(new ChangeListItem(group));
       }
     });
@@ -188,13 +197,18 @@ export class SvnCommands {
       return;
     }
 
-    let choice = picks[0];
-    if (picks.length > 1) {
-      const selectedChoice = await window.showQuickPick(picks, {});
-      if (!selectedChoice) {
-        return;
-      }
-      choice = selectedChoice;
+    let choice;
+    // If has only changes, not prompt to select changelist
+    if (picks.length === 1 && repository.changes.resourceStates.length) {
+      choice = picks[0];
+    } else {
+      choice = await window.showQuickPick(picks, {
+        placeHolder: "Select a changelist to commit"
+      });
+    }
+
+    if (!choice) {
+      return;
     }
 
     const filePaths = choice.resourceGroup.resourceStates.map(state => {