Drop obsolete members after release v0.17.0

11 files changed, 9 insertions, 279 deletions

diff --git a/LibGit2Sharp/CommitLog.cs b/LibGit2Sharp/CommitLog.cs
index a5306c8c..1056437a 100644
--- a/LibGit2Sharp/CommitLog.cs
+++ b/LibGit2Sharp/CommitLog.cs
@@ -82,29 +82,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Find the best possible common ancestor given two <see cref="Commit"/>s.
-        /// </summary>
-        /// <param name="first">The first <see cref="Commit"/>.</param>
-        /// <param name="second">The second <see cref="Commit"/>.</param>
-        /// <returns>The common ancestor or null if none found.</returns>
-        [Obsolete("This method will be removed in the next release. Please use FindMergeBase(Commit, Commit).")]
-        public Commit FindCommonAncestor(Commit first, Commit second)
-        {
-            return FindMergeBase(first, second);
-        }
-
-        /// <summary>
-        /// Find the best possible common ancestor given two or more <see cref="Commit"/>.
-        /// </summary>
-        /// <param name="commits">The <see cref="Commit"/>s for which to find the common ancestor.</param>
-        /// <returns>The common ancestor or null if none found.</returns>
-        [Obsolete("This method will be removed in the next release. Please use FindMergeBase(IEnumerable<Commit>, MergeBaseFindingStrategy).")]
-        public Commit FindCommonAncestor(IEnumerable<Commit> commits)
-        {
-            return FindMergeBase(commits, MergeBaseFindingStrategy.Octopus);
-        }
-
-        /// <summary>
         /// Find the best possible merge base given two <see cref="Commit"/>s.
         /// </summary>
         /// <param name="first">The first <see cref="Commit"/>.</param>
diff --git a/LibGit2Sharp/FetchHead.cs b/LibGit2Sharp/FetchHead.cs
index 7d5e6dfd..75f12ae0 100644
--- a/LibGit2Sharp/FetchHead.cs
+++ b/LibGit2Sharp/FetchHead.cs
@@ -7,8 +7,7 @@ namespace LibGit2Sharp
     /// Represents a local reference data from a remote repository which
     /// has been retreived through a Fetch process.
     /// </summary>
-    [Obsolete("This class is meant for internal use only and will not be public in the next release.")]
-    public class FetchHead : ReferenceWrapper<GitObject>
+    internal class FetchHead : ReferenceWrapper<GitObject>
     {
         /// <summary>
         /// Needed for mocking purposes.
diff --git a/LibGit2Sharp/IQueryableCommitLog.cs b/LibGit2Sharp/IQueryableCommitLog.cs
index d73a4307..7642d412 100644
--- a/LibGit2Sharp/IQueryableCommitLog.cs
+++ b/LibGit2Sharp/IQueryableCommitLog.cs
@@ -1,5 +1,4 @@
-using System;
-using System.Collections.Generic;
+using System.Collections.Generic;
 
 namespace LibGit2Sharp
 {
@@ -16,23 +15,6 @@ namespace LibGit2Sharp
         ICommitLog QueryBy(CommitFilter filter);
 
         /// <summary>
-        /// Find the best possible common ancestor given two <see cref="Commit"/>s.
-        /// </summary>
-        /// <param name="first">The first <see cref="Commit"/>.</param>
-        /// <param name="second">The second <see cref="Commit"/>.</param>
-        /// <returns>The common ancestor or null if none found.</returns>
-        [Obsolete("This method will be removed in the next release. Please use FindMergeBase(Commit, Commit).")]
-        Commit FindCommonAncestor(Commit first, Commit second);
-
-        /// <summary>
-        /// Find the best possible common ancestor given two or more <see cref="Commit"/>s.
-        /// </summary>
-        /// <param name="commits">The <see cref="Commit"/> for which to find the common ancestor.</param>
-        /// <returns>The common ancestor or null if none found.</returns>
-        [Obsolete("This method will be removed in the next release. Please use FindMergeBase(IEnumerable<Commit>, MergeBaseFindingStrategy).")]
-        Commit FindCommonAncestor(IEnumerable<Commit> commits);
-
-        /// <summary>
         /// Find the best possible merge base given two <see cref="Commit"/>s.
         /// </summary>
         /// <param name="first">The first <see cref="Commit"/>.</param>
diff --git a/LibGit2Sharp/IRepository.cs b/LibGit2Sharp/IRepository.cs
index 78f1a156..44e40d2a 100644
--- a/LibGit2Sharp/IRepository.cs
+++ b/LibGit2Sharp/IRepository.cs
@@ -168,19 +168,6 @@ namespace LibGit2Sharp
         Commit Commit(string message, Signature author, Signature committer, CommitOptions options = null);
 
         /// <summary>
-        /// Stores the content of the <see cref="Repository.Index"/> as a new <see cref="LibGit2Sharp.Commit"/> into the repository.
-        /// The tip of the <see cref="Repository.Head"/> will be used as the parent of this new Commit.
-        /// Once the commit is created, the <see cref="Repository.Head"/> will move forward to point at it.
-        /// </summary>
-        /// <param name="message">The description of why a change was made to the repository.</param>
-        /// <param name="author">The <see cref="Signature"/> of who made the change.</param>
-        /// <param name="committer">The <see cref="Signature"/> of who added the change to the repository.</param>
-        /// <param name="amendPreviousCommit">True to amend the current <see cref="LibGit2Sharp.Commit"/> pointed at by <see cref="Repository.Head"/>, false otherwise.</param>
-        /// <returns>The generated <see cref="LibGit2Sharp.Commit"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Please use a Commit overload that accepts a CommitOptions instead.")]
-        Commit Commit(string message, Signature author, Signature committer, bool amendPreviousCommit);
-
-        /// <summary>
         /// Sets the current <see cref="Head"/> to the specified commit and optionally resets the <see cref="Index"/> and
         /// the content of the working tree to match.
         /// </summary>
@@ -207,12 +194,6 @@ namespace LibGit2Sharp
         void RemoveUntrackedFiles();
 
         /// <summary>
-        /// Gets the references to the tips that are currently being merged.
-        /// </summary>
-        [Obsolete("This property is meant for internal use only and will not be public in the next release.")]
-        IEnumerable<MergeHead> MergeHeads { get; }
-
-        /// <summary>
         /// Merge changes from commit into the branch pointed at by HEAD..
         /// </summary>
         /// <param name="commit">The commit to merge into the branch pointed at by HEAD.</param>
diff --git a/LibGit2Sharp/MergeHead.cs b/LibGit2Sharp/MergeHead.cs
index 896b34aa..2ad50817 100644
--- a/LibGit2Sharp/MergeHead.cs
+++ b/LibGit2Sharp/MergeHead.cs
@@ -1,13 +1,11 @@
-using System;
-using System.Globalization;
+using System.Globalization;
 
 namespace LibGit2Sharp
 {
     /// <summary>
     /// A merge head is a parent for the next commit.
     /// </summary>
-    [Obsolete("This class is meant for internal use only and will not be public in the next release.")]
-    public class MergeHead : ReferenceWrapper<Commit>
+    internal class MergeHead : ReferenceWrapper<Commit>
     {
         /// <summary>
         /// Needed for mocking purposes.
diff --git a/LibGit2Sharp/Network.cs b/LibGit2Sharp/Network.cs
index 4f67b718..f982d0d9 100644
--- a/LibGit2Sharp/Network.cs
+++ b/LibGit2Sharp/Network.cs
@@ -349,8 +349,7 @@ namespace LibGit2Sharp
         /// <summary>
         /// The heads that have been updated during the last fetch.
         /// </summary>
-        [Obsolete("This property is meant for internal use only and will not be public in the next release.")]
-        public virtual IEnumerable<FetchHead> FetchHeads
+        internal virtual IEnumerable<FetchHead> FetchHeads
         {
             get
             {
diff --git a/LibGit2Sharp/ObjectDatabase.cs b/LibGit2Sharp/ObjectDatabase.cs
index c180d424..8f3e19ae 100644
--- a/LibGit2Sharp/ObjectDatabase.cs
+++ b/LibGit2Sharp/ObjectDatabase.cs
@@ -184,21 +184,6 @@ namespace LibGit2Sharp
 
         /// <summary>
         /// Inserts a <see cref="Commit"/> into the object database, referencing an existing <see cref="Tree"/>.
-        /// </summary>
-        /// <param name="message">The description of why a change was made to the repository.</param>
-        /// <param name="author">The <see cref="Signature"/> of who made the change.</param>
-        /// <param name="committer">The <see cref="Signature"/> of who added the change to the repository.</param>
-        /// <param name="tree">The <see cref="Tree"/> of the <see cref="Commit"/> to be created.</param>
-        /// <param name="parents">The parents of the <see cref="Commit"/> to be created.</param>
-        /// <returns>The created <see cref="Commit"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Please use CreateCommit(Signature, Signature, string, bool, Tree, IEnumerable<Commit>) instead.")]
-        public virtual Commit CreateCommit(string message, Signature author, Signature committer, Tree tree, IEnumerable<Commit> parents)
-        {
-            return CreateCommit(author, committer, message, true, tree, parents);
-        }
-
-        /// <summary>
-        /// Inserts a <see cref="Commit"/> into the object database, referencing an existing <see cref="Tree"/>.
         /// <para>
         /// Prettifing the message includes:
         /// * Removing empty lines from the beginning and end.
diff --git a/LibGit2Sharp/ReferenceCollection.cs b/LibGit2Sharp/ReferenceCollection.cs
index f7c60fc0..6ec43162 100644
--- a/LibGit2Sharp/ReferenceCollection.cs
+++ b/LibGit2Sharp/ReferenceCollection.cs
@@ -99,20 +99,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Creates a direct reference with the specified name and target
-        /// </summary>
-        /// <param name="name">The canonical name of the reference to create.</param>
-        /// <param name="targetId">Id of the target object.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> when adding the <see cref="DirectReference"/></param>
-        /// <param name="allowOverwrite">True to allow silent overwriting a potentially existing reference, false otherwise.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public virtual DirectReference Add(string name, ObjectId targetId, bool allowOverwrite, string logMessage)
-        {
-            return Add(name, targetId, null, logMessage, allowOverwrite);
-        }
-
-        /// <summary>
         /// Creates a symbolic reference with the specified name and target
         /// </summary>
         /// <param name="name">The canonical name of the reference to create.</param>
@@ -146,20 +132,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Creates a symbolic reference with the specified name and target
-        /// </summary>
-        /// <param name="name">The canonical name of the reference to create.</param>
-        /// <param name="targetRef">The target reference.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> when adding the <see cref="SymbolicReference"/></param>
-        /// <param name="allowOverwrite">True to allow silent overwriting a potentially existing reference, false otherwise.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public virtual SymbolicReference Add(string name, Reference targetRef, bool allowOverwrite, string logMessage)
-        {
-            return Add(name, targetRef, null, logMessage, allowOverwrite);
-        }
-
-        /// <summary>
         /// Remove a reference from the repository
         /// </summary>
         /// <param name="reference">The reference to delete.</param>
@@ -258,19 +230,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Updates the target of a direct reference
-        /// </summary>
-        /// <param name="directRef">The direct reference which target should be updated.</param>
-        /// <param name="targetId">The new target.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> of the <paramref name="directRef"/> reference</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public virtual Reference UpdateTarget(Reference directRef, ObjectId targetId, string logMessage)
-        {
-            return UpdateTarget(directRef, targetId, null, logMessage);
-        }
-
-        /// <summary>
         /// Updates the target of a symbolic reference
         /// </summary>
         /// <param name="symbolicRef">The symbolic reference which target should be updated.</param>
@@ -308,19 +267,6 @@ namespace LibGit2Sharp
             return UpdateTarget(symbolicRef, targetRef, null, null);
         }
 
-        /// <summary>
-        /// Updates the target of a symbolic reference
-        /// </summary>
-        /// <param name="symbolicRef">The symbolic reference which target should be updated.</param>
-        /// <param name="targetRef">The new target.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> of the <paramref name="symbolicRef"/> reference.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public virtual Reference UpdateTarget(Reference symbolicRef, Reference targetRef, string logMessage)
-        {
-            return UpdateTarget(symbolicRef, targetRef, null, logMessage);
-        }
-
         internal Reference UpdateHeadTarget<T>(T target, Signature signature, string logMessage)
         {
             Debug.Assert(signature != null);
diff --git a/LibGit2Sharp/ReferenceCollectionExtensions.cs b/LibGit2Sharp/ReferenceCollectionExtensions.cs
index ee552649..c0e65d6c 100644
--- a/LibGit2Sharp/ReferenceCollectionExtensions.cs
+++ b/LibGit2Sharp/ReferenceCollectionExtensions.cs
@@ -92,21 +92,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Creates a direct or symbolic reference with the specified name and target
-        /// </summary>
-        /// <param name="refsColl">The <see cref="ReferenceCollection"/> being worked with.</param>
-        /// <param name="name">The name of the reference to create.</param>
-        /// <param name="canonicalRefNameOrObjectish">The target which can be either the canonical name of a reference or a revparse spec.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> when adding the <see cref="Reference"/></param>
-        /// <param name="allowOverwrite">True to allow silent overwriting a potentially existing reference, false otherwise.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public static Reference Add(this ReferenceCollection refsColl, string name, string canonicalRefNameOrObjectish, bool allowOverwrite, string logMessage)
-        {
-            return Add(refsColl, name, canonicalRefNameOrObjectish, null, logMessage, allowOverwrite);
-        }
-
-        /// <summary>
         /// Updates the target of a direct reference.
         /// </summary>
         /// <param name="refsColl">The <see cref="ReferenceCollection"/> being worked with.</param>
@@ -140,20 +125,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Updates the target of a direct reference
-        /// </summary>
-        /// <param name="refsColl">The <see cref="ReferenceCollection"/> being worked with.</param>
-        /// <param name="directRef">The direct reference which target should be updated.</param>
-        /// <param name="objectish">The revparse spec of the target.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> of the <paramref name="directRef"/> reference.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public static Reference UpdateTarget(this ReferenceCollection refsColl, Reference directRef, string objectish, string logMessage)
-        {
-            return UpdateTarget(refsColl, directRef, objectish, null, logMessage);
-        }
-
-        /// <summary>
         /// Rename an existing reference with a new name
         /// </summary>
         /// <param name="currentName">The canonical name of the reference to rename.</param>
@@ -240,21 +211,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Updates the target of a reference
-        /// </summary>
-        /// <param name="refsColl">The <see cref="ReferenceCollection"/> being worked with.</param>
-        /// <param name="name">The canonical name of the reference.</param>
-        /// <param name="canonicalRefNameOrObjectish">The target which can be either the canonical name of a reference or a revparse spec.</param>
-        /// <param name="logMessage">The optional message to log in the <see cref="ReflogCollection"/> of the <paramref name="name"/> reference.</param>
-        /// <returns>A new <see cref="Reference"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Prefer the overload that takes a signature and a message for the reflog.")]
-        public static Reference UpdateTarget(this ReferenceCollection refsColl, string name, string canonicalRefNameOrObjectish, string logMessage)
-        {
-            return UpdateTarget(refsColl, name, canonicalRefNameOrObjectish, null, logMessage);
-        }
-
-
-        /// <summary>
         /// Delete a reference with the specified name
         /// </summary>
         /// <param name="refsColl">The <see cref="ReferenceCollection"/> being worked with.</param>
diff --git a/LibGit2Sharp/Repository.cs b/LibGit2Sharp/Repository.cs
index 5523fb6d..3a9bbbef 100644
--- a/LibGit2Sharp/Repository.cs
+++ b/LibGit2Sharp/Repository.cs
@@ -547,8 +547,10 @@ namespace LibGit2Sharp
         /// <param name="options"><see cref="CloneOptions"/> controlling clone behavior</param>
         /// <returns>The path to the created repository.</returns>
         public static string Clone(string sourceUrl, string workdirPath,
-            CloneOptions options)
+            CloneOptions options = null)
         {
+            options = options ?? new CloneOptions();
+
             CheckoutCallbacks checkoutCallbacks = CheckoutCallbacks.GenerateCheckoutCallbacks(
                 options.OnCheckoutProgress, null);
 
@@ -582,49 +584,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Clone with specified options.
-        /// </summary>
-        /// <param name="sourceUrl">URI for the remote repository</param>
-        /// <param name="workdirPath">Local path to clone into</param>
-        /// <param name="bare">True will result in a bare clone, false a full clone.</param>
-        /// <param name="checkout">If true, the origin's HEAD will be checked out. This only applies
-        /// to non-bare repositories.</param>
-        /// <param name="onTransferProgress">Handler for network transfer and indexing progress information</param>
-        /// <param name="onCheckoutProgress">Handler for checkout progress information</param>
-        /// <param name="credentials">Credentials to use for user/pass authentication</param>
-        /// <returns>The path to the created repository.</returns>
-        [Obsolete("This overload will be removed in the next release. Please use Repository.Clone(string, string, CloneOptions) instead.")]
-        public static string Clone(string sourceUrl, string workdirPath,
-            bool bare = false,
-            bool checkout = true,
-            TransferProgressHandler onTransferProgress = null,
-            CheckoutProgressHandler onCheckoutProgress = null,
-            Credentials credentials = null)
-        {
-            return Clone(sourceUrl, workdirPath, new CloneOptions()
-            {
-                IsBare = bare,
-                Checkout = checkout,
-                OnTransferProgress = onTransferProgress,
-                OnCheckoutProgress = onCheckoutProgress,
-                Credentials = credentials
-            });
-        }
-
-        /// <summary>
-        /// Clone without options.
-        /// </summary>
-        /// <param name="sourceUrl">URI for the remote repository</param>
-        /// <param name="workdirPath">Local path to clone into</param>
-        /// <returns>The path to the created repository.</returns>
-        public static string Clone(string sourceUrl, string workdirPath)
-        {
-            // This overload is required to supress the obsolete warning if called without arguments.
-            // Should be removed once the obsolete overload is removed.
-            return Clone(sourceUrl, workdirPath, new CloneOptions());
-        }
-
-        /// <summary>
         /// Find where each line of a file originated.
         /// </summary>
         /// <param name="path">Path of the file to blame.</param>
@@ -944,22 +903,6 @@ namespace LibGit2Sharp
             return result;
         }
 
-        /// <summary>
-        /// Stores the content of the <see cref="Repository.Index"/> as a new <see cref="LibGit2Sharp.Commit"/> into the repository.
-        /// The tip of the <see cref="Repository.Head"/> will be used as the parent of this new Commit.
-        /// Once the commit is created, the <see cref="Repository.Head"/> will move forward to point at it.
-        /// </summary>
-        /// <param name="message">The description of why a change was made to the repository.</param>
-        /// <param name="author">The <see cref="Signature"/> of who made the change.</param>
-        /// <param name="committer">The <see cref="Signature"/> of who added the change to the repository.</param>
-        /// <param name="amendPreviousCommit">True to amend the current <see cref="LibGit2Sharp.Commit"/> pointed at by <see cref="Repository.Head"/>, false otherwise.</param>
-        /// <returns>The generated <see cref="LibGit2Sharp.Commit"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Please use a Commit overload that accepts a CommitOptions instead.")]
-        public Commit Commit(string message, Signature author, Signature committer, bool amendPreviousCommit)
-        {
-            return Commit(message, author, committer, new CommitOptions { AmendPreviousCommit = amendPreviousCommit });
-        }
-
         private string BuildCommitLogMessage(Commit commit, bool amendPreviousCommit, bool isHeadOrphaned, bool isMergeCommit)
         {
             string kind = string.Empty;
@@ -1351,8 +1294,7 @@ namespace LibGit2Sharp
         /// <summary>
         /// Gets the references to the tips that are currently being merged.
         /// </summary>
-        [Obsolete("This property is meant for internal use only and will not be public in the next release.")]
-        public IEnumerable<MergeHead> MergeHeads
+        internal IEnumerable<MergeHead> MergeHeads
         {
             get
             {
diff --git a/LibGit2Sharp/RepositoryExtensions.cs b/LibGit2Sharp/RepositoryExtensions.cs
index 54981b2f..9f31d71e 100644
--- a/LibGit2Sharp/RepositoryExtensions.cs
+++ b/LibGit2Sharp/RepositoryExtensions.cs
@@ -217,24 +217,6 @@ namespace LibGit2Sharp
         /// Stores the content of the <see cref="Repository.Index"/> as a new <see cref="LibGit2Sharp.Commit"/> into the repository.
         /// The tip of the <see cref="Repository.Head"/> will be used as the parent of this new Commit.
         /// Once the commit is created, the <see cref="Repository.Head"/> will move forward to point at it.
-        /// <para>Both the Author and Committer will be guessed from the Git configuration. An exception will be raised if no configuration is reachable.</para>
-        /// </summary>
-        /// <param name="repository">The <see cref="Repository"/> being worked with.</param>
-        /// <param name="message">The description of why a change was made to the repository.</param>
-        /// <param name="amendPreviousCommit">True to amend the current <see cref="LibGit2Sharp.Commit"/> pointed at by <see cref="Repository.Head"/>, false otherwise.</param>
-        /// <returns>The generated <see cref="LibGit2Sharp.Commit"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Please use a Commit overload that accepts a CommitOptions instead.")]
-        public static Commit Commit(this IRepository repository, string message, bool amendPreviousCommit)
-        {
-            Signature author = repository.Config.BuildSignature(DateTimeOffset.Now, true);
-
-            return repository.Commit(message, author, new CommitOptions { AmendPreviousCommit = amendPreviousCommit });
-        }
-
-        /// <summary>
-        /// Stores the content of the <see cref="Repository.Index"/> as a new <see cref="LibGit2Sharp.Commit"/> into the repository.
-        /// The tip of the <see cref="Repository.Head"/> will be used as the parent of this new Commit.
-        /// Once the commit is created, the <see cref="Repository.Head"/> will move forward to point at it.
         /// <para>The Committer will be guessed from the Git configuration. An exception will be raised if no configuration is reachable.</para>
         /// </summary>
         /// <param name="repository">The <see cref="Repository"/> being worked with.</param>
@@ -250,23 +232,6 @@ namespace LibGit2Sharp
         }
 
         /// <summary>
-        /// Stores the content of the <see cref="Repository.Index"/> as a new <see cref="LibGit2Sharp.Commit"/> into the repository.
-        /// The tip of the <see cref="Repository.Head"/> will be used as the parent of this new Commit.
-        /// Once the commit is created, the <see cref="Repository.Head"/> will move forward to point at it.
-        /// <para>The Committer will be guessed from the Git configuration. An exception will be raised if no configuration is reachable.</para>
-        /// </summary>
-        /// <param name="repository">The <see cref="Repository"/> being worked with.</param>
-        /// <param name="author">The <see cref="Signature"/> of who made the change.</param>
-        /// <param name="message">The description of why a change was made to the repository.</param>
-        /// <param name="amendPreviousCommit">True to amend the current <see cref="LibGit2Sharp.Commit"/> pointed at by <see cref="Repository.Head"/>, false otherwise.</param>
-        /// <returns>The generated <see cref="LibGit2Sharp.Commit"/>.</returns>
-        [Obsolete("This method will be removed in the next release. Please use a Commit overload that accepts a CommitOptions instead.")]
-        public static Commit Commit(this IRepository repository, string message, Signature author, bool amendPreviousCommit)
-        {
-            return repository.Commit(message, author, new CommitOptions { AmendPreviousCommit = amendPreviousCommit });
-        }
-
-        /// <summary>
         /// Fetch from the specified remote.
         /// </summary>
         /// <param name="repository">The <see cref="Repository"/> being worked with.</param>