From f742d507d2f3bc7bbb3e3b8f017c806b70f5de38 Mon Sep 17 00:00:00 2001
From: half-ogre <nemo@half-ogre.com>
Date: Tue, 15 Oct 2013 11:46:35 -0700
Subject: [PATCH] add API doc for IApiConnection

---
 Octokit/Http/IApiConnection.cs | 85 +++++++++++++++++++++++++++++++++-
 1 file changed, 83 insertions(+), 2 deletions(-)

diff --git a/Octokit/Http/IApiConnection.cs b/Octokit/Http/IApiConnection.cs
index 86811bf9..15168231 100644
--- a/Octokit/Http/IApiConnection.cs
+++ b/Octokit/Http/IApiConnection.cs
@@ -7,20 +7,101 @@ using System.Threading.Tasks;
 namespace Octokit
 {
     /// <summary>
-    /// Provides type-friendly convenience methods the wrap <see cref="IConnection"/> methods.
+    /// A connection for making API requests against URI endpoints.
+    /// Provides type-friendly convenience methods that wrap <see cref="IConnection"/> methods.
     /// </summary>
     public interface IApiConnection
     {
+        /// <summary>
+        /// Gets the API resource at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">Type of the API resource to get.</typeparam>
+        /// <param name="uri">URI of the API resource to get.</param>
+        /// <param name="parameters">Parameters to add to the API request.</param>
+        /// <returns>The API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         [SuppressMessage("Microsoft.Naming", "CA1716:IdentifiersShouldNotMatchKeywords", MessageId = "Get",
             Justification = "It's fiiiine. It's fine. Trust us.")]
         Task<T> Get<T>(Uri uri, IDictionary<string, string> parameters);
+
+        /// <summary>
+        /// Gets the HTML content of the API resource at the specified URI.
+        /// </summary>
+        /// <param name="uri">URI of the API resource to get.</param>
+        /// <param name="parameters">Parameters to add to the API request.</param>
+        /// <returns>The API resource's HTML content.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<string> GetHtml(Uri uri, IDictionary<string, string> parameters);
+
+        /// <summary>
+        /// Gets all API resources in the list at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">Type of the API resource in the list.</typeparam>
+        /// <param name="uri">URI of the API resource to get.</param>
+        /// <param name="parameters">Parameters to add to the API request.</param>
+        /// <returns><see cref="IReadOnlyList{T}"/> of the The API resources in the list.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<IReadOnlyList<T>> GetAll<T>(Uri uri, IDictionary<string, string> parameters);
+
+        /// <summary>
+        /// Creates a new API resource in the list at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to get.</param>
+        /// <param name="data">Object that describes the new API resource; this will be serialized and used as the request's body.</param>
+        /// <returns>The created API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<T> Post<T>(Uri uri, object data);
-        Task<T> Post<T>(Uri uri, Stream rawData, string contentType, string accepts); 
+
+        /// <summary>
+        /// Creates a new API resource in the list at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to get.</param>
+        /// <param name="rawData">A <see cref="Stream"/> to use as the API request's body.</param>
+        /// <param name="contentType">Content type of the API request.</param>
+        /// <param name="accepts">Accept header to use for the API request.</param>
+        /// <returns>The created API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
+        Task<T> Post<T>(Uri uri, Stream rawData, string contentType, string accepts);
+
+        /// <summary>
+        /// Creates or replaces the API resource at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to create or replace.</param>
+        /// <param name="data">Object that describes the API resource; this will be serialized and used as the request's body.</param>
+        /// <returns>The created API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<T> Put<T>(Uri uri, object data);
+
+        /// <summary>
+        /// Creates or replaces the API resource at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to create or replace.</param>
+        /// <param name="data">Object that describes the API resource; this will be serialized and used as the request's body.</param>
+        /// <param name="twoFactorAuthenticationCode">The two-factor authentication code in response to the current user's previous challenge.</param>
+        /// <returns>The created API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<T> Put<T>(Uri uri, object data, string twoFactorAuthenticationCode);
+
+        /// <summary>
+        /// Updates the API resource at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to update.</param>
+        /// /// <param name="data">Object that describes the API resource; this will be serialized and used as the request's body.</param>
+        /// <returns>The updated API resource.</returns>
+        /// <exception cref="ApiException">Thrown when an API error occurs.</exception>
         Task<T> Patch<T>(Uri uri, object data);
+
+        /// <summary>
+        /// Deletes the API object at the specified URI.
+        /// </summary>
+        /// <typeparam name="T">The API resource's type.</typeparam>
+        /// <param name="uri">URI of the API resource to delete.</param>
+        /// <returns>A <see cref="Task"/> for the request's execution.</returns>
         [SuppressMessage("Microsoft.Design", "CA1004:GenericMethodsShouldProvideTypeParameter", Justification="Legitimate, but I'm not fixing it just yet.")]
         Task Delete<T>(Uri uri);
     }