From 4230c958f8d042918c7f94b1bcca3cfdac1d5db9 Mon Sep 17 00:00:00 2001
From: "aedampir@gmail.com" <aedampir@gmail.com>
Date: Fri, 10 Jun 2016 14:52:29 +0700
Subject: [PATCH] modified XML docs

---
 .../IObservableRepositoryHooksClient.cs       |  64 +++++++----
 .../ObservableRepositoryHooksClient.cs        | 100 +++++++++++-------
 Octokit/Clients/IRepositoryHooksClient.cs     |  62 +++++++----
 Octokit/Clients/RepositoryHooksClient.cs      |  96 ++++++++++-------
 4 files changed, 207 insertions(+), 115 deletions(-)

diff --git a/Octokit.Reactive/Clients/IObservableRepositoryHooksClient.cs b/Octokit.Reactive/Clients/IObservableRepositoryHooksClient.cs
index 53d2f592..e68ce767 100644
--- a/Octokit.Reactive/Clients/IObservableRepositoryHooksClient.cs
+++ b/Octokit.Reactive/Clients/IObservableRepositoryHooksClient.cs
@@ -4,73 +4,95 @@ using System.Reactive;
 
 namespace Octokit.Reactive
 {
+    /// <summary>
+    /// A client for GitHub's Repository Webhooks API.
+    /// </summary>
+    /// <remarks>
+    /// See the <a href="https://developer.github.com/v3/repos/hooks/">Webhooks API documentation</a> for more information.
+    /// </remarks>
     public interface IObservableRepositoryHooksClient
     {
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
-        /// <returns></returns>
-        IObservable<RepositoryHook> GetAll(string owner, string repositoryName);
+        /// <param name="name">The repository's name</param>
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IObservable{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        IObservable<RepositoryHook> GetAll(string owner, string name);
 
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
+        /// <param name="name">The repository's name</param>
         /// <param name="options">Options for changing the API response</param>
-        /// <returns></returns>
-        IObservable<RepositoryHook> GetAll(string owner, string repositoryName, ApiOptions options);
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IObservable{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        IObservable<RepositoryHook> GetAll(string owner, string name, ApiOptions options);
 
         /// <summary>
-        /// Gets a single hook defined for a repository by id
+        /// Gets a single hook by Id
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#get-single-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
+        /// <returns>A <see cref="RepositoryHook"/> representing hook for specified hook id</returns>
         [SuppressMessage("Microsoft.Naming", "CA1716:IdentifiersShouldNotMatchKeywords", MessageId = "Get", Justification = "This is ok; we're matching HTTP verbs not keyworks")]
-        IObservable<RepositoryHook> Get(string owner, string repositoryName, int hookId);
+        IObservable<RepositoryHook> Get(string owner, string name, int hookId);
 
         /// <summary>
         /// Creates a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hook">The hook's parameters</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#create-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        IObservable<RepositoryHook> Create(string owner, string repositoryName, NewRepositoryHook hook);
+        /// <returns>A <see cref="RepositoryHook"/> representing created hook for specified repository</returns>
+        IObservable<RepositoryHook> Create(string owner, string name, NewRepositoryHook hook);
 
         /// <summary>
         /// Edits a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
+        /// <param name="hook">The requested changes to an edit repository hook</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        IObservable<RepositoryHook> Edit(string owner, string repositoryName, int hookId, EditRepositoryHook hook);
+        /// <returns>A <see cref="RepositoryHook"/> representing modified hook for specified repository</returns>
+        IObservable<RepositoryHook> Edit(string owner, string name, int hookId, EditRepositoryHook hook);
 
         /// <summary>
         /// Tests a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#test-a-hook">API documentation</a> for more information. 
         /// This will trigger the hook with the latest push to the current repository if the hook is subscribed to push events. If the hook 
         /// is not subscribed to push events, the server will respond with 204 but no test POST will be generated.</remarks>
         /// <returns></returns>
-        IObservable<Unit> Test(string owner, string repositoryName, int hookId);
+        IObservable<Unit> Test(string owner, string name, int hookId);
 
         /// <summary>
         /// This will trigger a ping event to be sent to the hook.
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        IObservable<Unit> Ping(string owner, string repositoryName, int hookId);
+        IObservable<Unit> Ping(string owner, string name, int hookId);
 
         /// <summary>
         /// Deletes a hook for a repository
         /// </summary>
-        /// <param name="owner"></param>
-        /// <param name="repositoryName"></param>
-        /// <param name="hookId"></param>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#delete-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        IObservable<Unit> Delete(string owner, string repositoryName, int hookId);
+        IObservable<Unit> Delete(string owner, string name, int hookId);
     }
 }
\ No newline at end of file
diff --git a/Octokit.Reactive/Clients/ObservableRepositoryHooksClient.cs b/Octokit.Reactive/Clients/ObservableRepositoryHooksClient.cs
index 24bca1f4..78585ce1 100644
--- a/Octokit.Reactive/Clients/ObservableRepositoryHooksClient.cs
+++ b/Octokit.Reactive/Clients/ObservableRepositoryHooksClient.cs
@@ -5,11 +5,21 @@ using Octokit.Reactive.Internal;
 
 namespace Octokit.Reactive
 {
+    /// <summary>
+    /// A client for GitHub's Repository Webhooks API.
+    /// </summary>
+    /// <remarks>
+    /// See the <a href="https://developer.github.com/v3/repos/hooks/">Webhooks API documentation</a> for more information.
+    /// </remarks>
     public class ObservableRepositoryHooksClient : IObservableRepositoryHooksClient
     {
         readonly IRepositoryHooksClient _client;
         readonly IConnection _connection;
 
+        /// <summary>
+        /// Initializes a new GitHub Webhooks API client.
+        /// </summary>
+        /// <param name="client"></param>
         public ObservableRepositoryHooksClient(IGitHubClient client)
         {
             Ensure.ArgumentNotNull(client, "client");
@@ -21,118 +31,134 @@ namespace Octokit.Reactive
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
-        /// <returns></returns>
-        public IObservable<RepositoryHook> GetAll(string owner, string repositoryName)
+        /// <param name="name">The repository's name</param>
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IObservable{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        public IObservable<RepositoryHook> GetAll(string owner, string name)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return GetAll(owner, repositoryName, ApiOptions.None);
+            return GetAll(owner, name, ApiOptions.None);
         }
 
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
+        /// <param name="name">The repository's name</param>
         /// <param name="options">Options for changing the API response</param>
-        /// <returns></returns>
-        public IObservable<RepositoryHook> GetAll(string owner, string repositoryName, ApiOptions options)
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IObservable{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        public IObservable<RepositoryHook> GetAll(string owner, string name, ApiOptions options)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(options, "options");
 
-            return _connection.GetAndFlattenAllPages<RepositoryHook>(ApiUrls.RepositoryHooks(owner, repositoryName), options);
+            return _connection.GetAndFlattenAllPages<RepositoryHook>(ApiUrls.RepositoryHooks(owner, name), options);
         }
 
         /// <summary>
-        /// Gets a single hook defined for a repository by id
+        /// Gets a single hook by Id
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#get-single-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        public IObservable<RepositoryHook> Get(string owner, string repositoryName, int hookId)
+        /// <returns>A <see cref="RepositoryHook"/> representing hook for specified hook id</returns>
+        public IObservable<RepositoryHook> Get(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return _client.Get(owner, repositoryName, hookId).ToObservable();
+            return _client.Get(owner, name, hookId).ToObservable();
         }
 
         /// <summary>
         /// Creates a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hook">The hook's parameters</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#create-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        public IObservable<RepositoryHook> Create(string owner, string repositoryName, NewRepositoryHook hook)
+        /// <returns>A <see cref="RepositoryHook"/> representing created hook for specified repository</returns>
+        public IObservable<RepositoryHook> Create(string owner, string name, NewRepositoryHook hook)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(hook, "hook");
 
-            return _client.Create(owner, repositoryName, hook).ToObservable();
+            return _client.Create(owner, name, hook).ToObservable();
         }
 
         /// <summary>
         /// Edits a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
+        /// <param name="hook">The requested changes to an edit repository hook</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        public IObservable<RepositoryHook> Edit(string owner, string repositoryName, int hookId, EditRepositoryHook hook)
+        /// <returns>A <see cref="RepositoryHook"/> representing modified hook for specified repository</returns>
+        public IObservable<RepositoryHook> Edit(string owner, string name, int hookId, EditRepositoryHook hook)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(hook, "hook");
 
-            return _client.Edit(owner, repositoryName, hookId, hook).ToObservable();
+            return _client.Edit(owner, name, hookId, hook).ToObservable();
         }
 
         /// <summary>
         /// Tests a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#test-a-hook">API documentation</a> for more information. 
         /// This will trigger the hook with the latest push to the current repository if the hook is subscribed to push events. If the hook 
         /// is not subscribed to push events, the server will respond with 204 but no test POST will be generated.</remarks>
         /// <returns></returns>
-        public IObservable<Unit> Test(string owner, string repositoryName, int hookId)
+        public IObservable<Unit> Test(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return _client.Test(owner, repositoryName, hookId).ToObservable();
+            return _client.Test(owner, name, hookId).ToObservable();
         }
 
         /// <summary>
         /// This will trigger a ping event to be sent to the hook.
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        public IObservable<Unit> Ping(string owner, string repositoryName, int hookId)
+        public IObservable<Unit> Ping(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return _client.Ping(owner, repositoryName, hookId).ToObservable();
+            return _client.Ping(owner, name, hookId).ToObservable();
         }
 
         /// <summary>
         /// Deletes a hook for a repository
         /// </summary>
-        /// <param name="owner"></param>
-        /// <param name="repositoryName"></param>
-        /// <param name="hookId"></param>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#delete-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        public IObservable<Unit> Delete(string owner, string repositoryName, int hookId)
+        public IObservable<Unit> Delete(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return _client.Delete(owner, repositoryName, hookId).ToObservable();
+            return _client.Delete(owner, name, hookId).ToObservable();
         }
     }
 }
diff --git a/Octokit/Clients/IRepositoryHooksClient.cs b/Octokit/Clients/IRepositoryHooksClient.cs
index 571306a4..039406f1 100644
--- a/Octokit/Clients/IRepositoryHooksClient.cs
+++ b/Octokit/Clients/IRepositoryHooksClient.cs
@@ -4,73 +4,95 @@ using System.Threading.Tasks;
 
 namespace Octokit
 {
+    /// <summary>
+    /// A client for GitHub's Repository Webhooks API.
+    /// </summary>
+    /// <remarks>
+    /// See the <a href="https://developer.github.com/v3/repos/hooks/">Webhooks API documentation</a> for more information.
+    /// </remarks>
     public interface IRepositoryHooksClient
     {
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
-        /// <returns></returns>
-        Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string repositoryName);
+        /// <param name="name">The repository's name</param>
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IReadOnlyList{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string name);
 
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
+        /// <param name="name">The repository's name</param>
         /// <param name="options">Options for changing the API response</param>
-        /// <returns></returns>
-        Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string repositoryName, ApiOptions options);
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IReadOnlyList{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string name, ApiOptions options);
 
         /// <summary>
         /// Gets a single hook by Id
         /// </summary>
-        /// <param name="owner"></param>
-        /// <param name="repositoryName"></param>
-        /// <param name="hookId"></param>
-        /// <returns></returns>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#get-single-hook">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="RepositoryHook"/> representing hook for specified hook id</returns>
         [SuppressMessage("Microsoft.Naming", "CA1716:IdentifiersShouldNotMatchKeywords", MessageId = "Get", Justification = "This is ok; we're matching HTTP verbs not keywords")]
-        Task<RepositoryHook> Get(string owner, string repositoryName, int hookId);
+        Task<RepositoryHook> Get(string owner, string name, int hookId);
 
         /// <summary>
         /// Creates a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hook">The hook's parameters</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#create-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        Task<RepositoryHook> Create(string owner, string repositoryName, NewRepositoryHook hook);
+        /// <returns>A <see cref="RepositoryHook"/> representing created hook for specified repository</returns>
+        Task<RepositoryHook> Create(string owner, string name, NewRepositoryHook hook);
 
         /// <summary>
         /// Edits a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
+        /// <param name="hook">The requested changes to an edit repository hook</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        Task<RepositoryHook> Edit(string owner, string repositoryName, int hookId, EditRepositoryHook hook);
+        /// <returns>A <see cref="RepositoryHook"/> representing modified hook for specified repository</returns>
+        Task<RepositoryHook> Edit(string owner, string name, int hookId, EditRepositoryHook hook);
 
         /// <summary>
         /// Tests a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#test-a-hook">API documentation</a> for more information. 
         /// This will trigger the hook with the latest push to the current repository if the hook is subscribed to push events. If the hook 
         /// is not subscribed to push events, the server will respond with 204 but no test POST will be generated.</remarks>
         /// <returns></returns>
-        Task Test(string owner, string repositoryName, int hookId);
+        Task Test(string owner, string name, int hookId);
 
         /// <summary>
         /// This will trigger a ping event to be sent to the hook.
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        Task Ping(string owner, string repositoryName, int hookId);
+        Task Ping(string owner, string name, int hookId);
 
         /// <summary>
         /// Deletes a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#delete-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        Task Delete(string owner, string repositoryName, int hookId);
+        Task Delete(string owner, string name, int hookId);
     }
 }
diff --git a/Octokit/Clients/RepositoryHooksClient.cs b/Octokit/Clients/RepositoryHooksClient.cs
index f2e6e9df..87a52177 100644
--- a/Octokit/Clients/RepositoryHooksClient.cs
+++ b/Octokit/Clients/RepositoryHooksClient.cs
@@ -3,10 +3,16 @@ using System.Threading.Tasks;
 
 namespace Octokit
 {
+    /// <summary>
+    /// A client for GitHub's Repository Webhooks API.
+    /// </summary>
+    /// <remarks>
+    /// See the <a href="https://developer.github.com/v3/repos/hooks/">Webhooks API documentation</a> for more information.
+    /// </remarks>
     public class RepositoryHooksClient : ApiClient, IRepositoryHooksClient
     {
         /// <summary>
-        /// Initializes a new GitHub Repos API client.
+        /// Initializes a new GitHub Webhooks API client.
         /// </summary>
         /// <param name="apiConnection">An API connection.</param>
         public RepositoryHooksClient(IApiConnection apiConnection)
@@ -17,118 +23,134 @@ namespace Octokit
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
-        /// <returns></returns>
-        public Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string repositoryName)
+        /// <param name="name">The repository's name</param>
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IReadOnlyList{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        public Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string name)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return GetAll(owner, repositoryName, ApiOptions.None);
+            return GetAll(owner, name, ApiOptions.None);
         }
 
         /// <summary>
         /// Gets the list of hooks defined for a repository
         /// </summary>
-        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
         /// <param name="owner">The repository's owner</param>
-        /// <param name="repositoryName">The repository's name</param>
+        /// <param name="name">The repository's name</param>
         /// <param name="options">Options for changing the API response</param>
-        /// <returns></returns>
-        public Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string repositoryName, ApiOptions options)
+        /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#list">API documentation</a> for more information.</remarks>
+        /// <returns>A <see cref="IReadOnlyList{RepositoryHook}"/> of <see cref="RepositoryHook"/>s representing hooks for specified repository</returns>
+        public Task<IReadOnlyList<RepositoryHook>> GetAll(string owner, string name, ApiOptions options)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(options, "options");
 
-            return ApiConnection.GetAll<RepositoryHook>(ApiUrls.RepositoryHooks(owner, repositoryName), options);
+            return ApiConnection.GetAll<RepositoryHook>(ApiUrls.RepositoryHooks(owner, name), options);
         }
 
         /// <summary>
         /// Gets a single hook by Id
         /// </summary>
-        /// <param name="owner"></param>
-        /// <param name="repositoryName"></param>
-        /// <param name="hookId"></param>
-        /// <returns></returns>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#get-single-hook">API documentation</a> for more information.</remarks>
-        public Task<RepositoryHook> Get(string owner, string repositoryName, int hookId)
+        /// <returns>A <see cref="RepositoryHook"/> representing hook for specified hook id</returns>
+        public Task<RepositoryHook> Get(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return ApiConnection.Get<RepositoryHook>(ApiUrls.RepositoryHookById(owner, repositoryName, hookId));
+            return ApiConnection.Get<RepositoryHook>(ApiUrls.RepositoryHookById(owner, name, hookId));
         }
 
         /// <summary>
         /// Creates a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hook">The hook's parameters</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#create-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        public Task<RepositoryHook> Create(string owner, string repositoryName, NewRepositoryHook hook)
+        /// <returns>A <see cref="RepositoryHook"/> representing created hook for specified repository</returns>
+        public Task<RepositoryHook> Create(string owner, string name, NewRepositoryHook hook)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(hook, "hook");
 
-            return ApiConnection.Post<RepositoryHook>(ApiUrls.RepositoryHooks(owner, repositoryName), hook.ToRequest());
+            return ApiConnection.Post<RepositoryHook>(ApiUrls.RepositoryHooks(owner, name), hook.ToRequest());
         }
 
         /// <summary>
         /// Edits a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
+        /// <param name="hook">The requested changes to an edit repository hook</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
-        /// <returns></returns>
-        public Task<RepositoryHook> Edit(string owner, string repositoryName, int hookId, EditRepositoryHook hook)
+        /// <returns>A <see cref="RepositoryHook"/> representing modified hook for specified repository</returns>
+        public Task<RepositoryHook> Edit(string owner, string name, int hookId, EditRepositoryHook hook)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
             Ensure.ArgumentNotNull(hook, "hook");
 
-            return ApiConnection.Patch<RepositoryHook>(ApiUrls.RepositoryHookById(owner, repositoryName, hookId), hook);
+            return ApiConnection.Patch<RepositoryHook>(ApiUrls.RepositoryHookById(owner, name, hookId), hook);
         }
 
         /// <summary>
         /// Tests a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#test-a-hook">API documentation</a> for more information. 
         /// This will trigger the hook with the latest push to the current repository if the hook is subscribed to push events. If the hook 
         /// is not subscribed to push events, the server will respond with 204 but no test POST will be generated.</remarks>
         /// <returns></returns>
-        public Task Test(string owner, string repositoryName, int hookId)
+        public Task Test(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return ApiConnection.Post(ApiUrls.RepositoryHookTest(owner, repositoryName, hookId));
+            return ApiConnection.Post(ApiUrls.RepositoryHookTest(owner, name, hookId));
         }
 
         /// <summary>
         /// This will trigger a ping event to be sent to the hook.
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#edit-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        public Task Ping(string owner, string repositoryName, int hookId)
+        public Task Ping(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return ApiConnection.Post(ApiUrls.RepositoryHookPing(owner, repositoryName, hookId));
+            return ApiConnection.Post(ApiUrls.RepositoryHookPing(owner, name, hookId));
         }
 
         /// <summary>
         /// Deletes a hook for a repository
         /// </summary>
+        /// <param name="owner">The repository's owner</param>
+        /// <param name="name">The repository's name</param>
+        /// <param name="hookId">The repository's hook id</param>
         /// <remarks>See <a href="http://developer.github.com/v3/repos/hooks/#delete-a-hook">API documentation</a> for more information.</remarks>
         /// <returns></returns>
-        public Task Delete(string owner, string repositoryName, int hookId)
+        public Task Delete(string owner, string name, int hookId)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, "owner");
-            Ensure.ArgumentNotNullOrEmptyString(repositoryName, "repositoryName");
+            Ensure.ArgumentNotNullOrEmptyString(name, "name");
 
-            return ApiConnection.Delete(ApiUrls.RepositoryHookById(owner, repositoryName, hookId));
+            return ApiConnection.Delete(ApiUrls.RepositoryHookById(owner, name, hookId));
         }
     }
 }
\ No newline at end of file