Drop optional parameters in BranchCollectionExtensions.cs

Author: shiftkey

LibGit2Sharp/BranchCollectionExtensions.cs

@@ -7,6 +7,18 @@ namespace LibGit2Sharp
     /// </summary>
     public static class BranchCollectionExtensions
     {
+        /// <summary>
+        /// Create a new local branch with the specified name
+        /// </summary>
+        /// <param name="branches">The <see cref="BranchCollection"/> being worked with.</param>
+        /// <param name="name">The name of the branch.</param>
+        /// <param name="commit">The target commit.</param>
+        /// <returns>A new <see cref="Branch"/>.</returns>
+        public static Branch Add(this BranchCollection branches, string name, Commit commit)
+        {
+            return branches.Add(name, commit, false);
+        }
+
         /// <summary>
         /// Create a new local branch with the specified name
         /// </summary>
@@ -15,21 +27,30 @@ public static class BranchCollectionExtensions
         /// <param name="commit">The target commit.</param>
         /// <param name="allowOverwrite">True to allow silent overwriting a potentially existing branch, false otherwise.</param>
         /// <returns>A new <see cref="Branch"/>.</returns>
-        public static Branch Add(this BranchCollection branches, string name, Commit commit, bool allowOverwrite = false)
+        public static Branch Add(this BranchCollection branches, string name, Commit commit, bool allowOverwrite)
         {
             Ensure.ArgumentNotNull(commit, "commit");
 
             return branches.Add(name, commit.Sha, allowOverwrite);
         }
 
+        /// <summary>
+        /// Deletes the branch with the specified name.
+        /// </summary>
+        /// <param name="name">The name of the branch to delete.</param>
+        /// <param name="branches">The <see cref="BranchCollection"/> being worked with.</param>
+        public static void Remove(this BranchCollection branches, string name)
+        {
+           branches.Remove(name, false);
+        }
 
         /// <summary>
         /// Deletes the branch with the specified name.
         /// </summary>
         /// <param name="name">The name of the branch to delete.</param>
         /// <param name="isRemote">True if the provided <paramref name="name"/> is the name of a remote branch, false otherwise.</param>
         /// <param name="branches">The <see cref="BranchCollection"/> being worked with.</param>
-        public static void Remove(this BranchCollection branches, string name, bool isRemote = false)
+        public static void Remove(this BranchCollection branches, string name, bool isRemote)
         {
             Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
@@ -45,6 +66,18 @@ public static void Remove(this BranchCollection branches, string name, bool isRe
             branches.Remove(branch);
         }
 
+        /// <summary>
+        /// Rename an existing local branch, using the default reflog message
+        /// </summary>
+        /// <param name="currentName">The current branch name.</param>
+        /// <param name="newName">The new name the existing branch should bear.</param>
+        /// <param name="branches">The <see cref="BranchCollection"/> being worked with.</param>
+        /// <returns>A new <see cref="Branch"/>.</returns>
+        public static Branch Rename(this BranchCollection branches, string currentName, string newName)
+        {
+            return branches.Rename(currentName, newName, false);
+        }
+
         /// <summary>
         /// Rename an existing local branch, using the default reflog message
         /// </summary>
@@ -53,7 +86,7 @@ public static void Remove(this BranchCollection branches, string name, bool isRe
         /// <param name="allowOverwrite">True to allow silent overwriting a potentially existing branch, false otherwise.</param>
         /// <param name="branches">The <see cref="BranchCollection"/> being worked with.</param>
         /// <returns>A new <see cref="Branch"/>.</returns>
-        public static Branch Rename(this BranchCollection branches, string currentName, string newName, bool allowOverwrite = false)
+        public static Branch Rename(this BranchCollection branches, string currentName, string newName, bool allowOverwrite)
         {
             Ensure.ArgumentNotNullOrEmptyString(currentName, "currentName");
             Ensure.ArgumentNotNullOrEmptyString(newName, "newName");