From 728fc798903e161da34ab0068db132ead6db363c Mon Sep 17 00:00:00 2001
From: Jozef Izso <jozef.izso@gmail.com>
Date: Sun, 13 May 2018 00:49:24 +0200
Subject: [PATCH 01/10] Implement new attributes for labels

---
 .../Clients/IssuesLabelsClientTests.cs           |  6 ++++--
 Octokit.Tests/Clients/IssuesLabelsClientTests.cs |  8 ++++----
 Octokit/Clients/IssuesLabelsClient.cs            |  8 ++++----
 Octokit/Helpers/AcceptHeaders.cs                 |  2 ++
 Octokit/Models/Request/LabelUpdate.cs            |  9 ++++++++-
 Octokit/Models/Request/NewLabel.cs               | 16 +++++++++++++++-
 Octokit/Models/Response/Label.cs                 | 14 +++++++++++++-
 7 files changed, 50 insertions(+), 13 deletions(-)

diff --git a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
index 59c5003818..c6d148c4fc 100644
--- a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
@@ -29,7 +29,7 @@ public IssuesLabelsClientTests()
     public async Task CanListIssueLabelsForAnIssue()
     {
         var newIssue = new NewIssue("A test issue") { Body = "A new unassigned issue" };
-        var newLabel = new NewLabel("test label", "FFFFFF");
+        var newLabel = new NewLabel("test label", "FFFFFF", "Test label description.");
 
         var label = await _issuesLabelsClient.Create(_context.RepositoryOwner, _context.RepositoryName, newLabel);
         var issue = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue);
@@ -46,13 +46,14 @@ public async Task CanListIssueLabelsForAnIssue()
 
         Assert.Equal(1, issueLabelsInfo.Count);
         Assert.Equal(newLabel.Color, issueLabelsInfo[0].Color);
+        Assert.Equal(newLabel.Description, issueLabelsInfo[0].Description);
     }
 
     [IntegrationTest]
     public async Task CanListIssueLabelsForAnIssueWithRepositoryId()
     {
         var newIssue = new NewIssue("A test issue") { Body = "A new unassigned issue" };
-        var newLabel = new NewLabel("test label", "FFFFFF");
+        var newLabel = new NewLabel("test label", "FFFFFF", "Test label description.");
 
         var label = await _issuesLabelsClient.Create(_context.Repository.Id, newLabel);
         var issue = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue);
@@ -69,6 +70,7 @@ public async Task CanListIssueLabelsForAnIssueWithRepositoryId()
 
         Assert.Equal(1, issueLabelsInfo.Count);
         Assert.Equal(newLabel.Color, issueLabelsInfo[0].Color);
+        Assert.Equal(newLabel.Description, issueLabelsInfo[0].Description);
     }
 
     [IntegrationTest]
diff --git a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
index 786cf1201d..b651723cad 100644
--- a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
@@ -520,7 +520,7 @@ public void CreatesCorrectUrl()
 
                 client.Create("fake", "repo", newLabel);
 
-                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), newLabel);
+                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), newLabel, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -533,7 +533,7 @@ public void CreatesCorrectUrlWithRepositoryId()
 
                 client.Create(1, newLabel);
 
-                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), newLabel);
+                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), newLabel, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -565,7 +565,7 @@ public void UpdatesCorrectUrl()
 
                 client.Update("fake", "repo", "labelName", labelUpdate);
 
-                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels/labelName"), labelUpdate);
+                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels/labelName"), labelUpdate, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -578,7 +578,7 @@ public void UpdatesCorrectUrlWithRepositoryId()
 
                 client.Update(1, "labelName", labelUpdate);
 
-                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels/labelName"), labelUpdate);
+                connection.Received().Post<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels/labelName"), labelUpdate, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
diff --git a/Octokit/Clients/IssuesLabelsClient.cs b/Octokit/Clients/IssuesLabelsClient.cs
index 964d567516..86a1a94f17 100644
--- a/Octokit/Clients/IssuesLabelsClient.cs
+++ b/Octokit/Clients/IssuesLabelsClient.cs
@@ -288,7 +288,7 @@ public Task<Label> Create(string owner, string name, NewLabel newLabel)
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(newLabel, nameof(newLabel));
 
-            return ApiConnection.Post<Label>(ApiUrls.Labels(owner, name), newLabel);
+            return ApiConnection.Post<Label>(ApiUrls.Labels(owner, name), newLabel, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -303,7 +303,7 @@ public Task<Label> Create(long repositoryId, NewLabel newLabel)
         {
             Ensure.ArgumentNotNull(newLabel, nameof(newLabel));
 
-            return ApiConnection.Post<Label>(ApiUrls.Labels(repositoryId), newLabel);
+            return ApiConnection.Post<Label>(ApiUrls.Labels(repositoryId), newLabel, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -323,7 +323,7 @@ public Task<Label> Update(string owner, string name, string labelName, LabelUpda
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
             Ensure.ArgumentNotNull(labelUpdate, nameof(labelUpdate));
 
-            return ApiConnection.Post<Label>(ApiUrls.Label(owner, name, labelName), labelUpdate);
+            return ApiConnection.Post<Label>(ApiUrls.Label(owner, name, labelName), labelUpdate, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -340,7 +340,7 @@ public Task<Label> Update(long repositoryId, string labelName, LabelUpdate label
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
             Ensure.ArgumentNotNull(labelUpdate, nameof(labelUpdate));
 
-            return ApiConnection.Post<Label>(ApiUrls.Label(repositoryId, labelName), labelUpdate);
+            return ApiConnection.Post<Label>(ApiUrls.Label(repositoryId, labelName), labelUpdate, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
diff --git a/Octokit/Helpers/AcceptHeaders.cs b/Octokit/Helpers/AcceptHeaders.cs
index b3701acacd..c096074e44 100644
--- a/Octokit/Helpers/AcceptHeaders.cs
+++ b/Octokit/Helpers/AcceptHeaders.cs
@@ -59,6 +59,8 @@ public static class AcceptHeaders
 
         public const string PreReceiveEnvironmentsPreview = "application/vnd.github.eye-scream-preview+json";
 
+        public const string LabelsApiPreview = "application/vnd.github.symmetra-preview+json";
+
         /// <summary>
         /// Combines multiple preview headers. GitHub API supports Accept header with multiple
         /// values separated by comma.
diff --git a/Octokit/Models/Request/LabelUpdate.cs b/Octokit/Models/Request/LabelUpdate.cs
index bcfbbfe151..b9f72afea0 100644
--- a/Octokit/Models/Request/LabelUpdate.cs
+++ b/Octokit/Models/Request/LabelUpdate.cs
@@ -18,13 +18,15 @@ public class LabelUpdate
         /// </summary>
         /// <param name="name">The name of the label.</param>
         /// <param name="color">The color of the label.</param>
-        public LabelUpdate(string name, string color)
+        /// <param name="description">The description of the label.</param>
+        public LabelUpdate(string name, string color, string description = null)
         {
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(color, nameof(color));
 
             Name = name;
             Color = color;
+            Description = description;
         }
 
         /// <summary>
@@ -49,6 +51,11 @@ public string Color
             }
         }
 
+        /// <summary>
+        /// A short description of the label (optional).
+        /// </summary>
+        public string Description { get; set; }
+
         internal string DebuggerDisplay
         {
             get
diff --git a/Octokit/Models/Request/NewLabel.cs b/Octokit/Models/Request/NewLabel.cs
index 2f6e596e26..83cdbae660 100644
--- a/Octokit/Models/Request/NewLabel.cs
+++ b/Octokit/Models/Request/NewLabel.cs
@@ -18,23 +18,32 @@ public class NewLabel
         /// </summary>
         /// <param name="name">The name of the label.</param>
         /// <param name="color">The color of the label.</param>
-        public NewLabel(string name, string color)
+        /// <param name="description">The description of the label.</param>
+        public NewLabel(string name, string color, string description = null)
         {
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(color, nameof(color));
 
             Name = name;
             Color = color;
+            Description = description;
         }
 
         /// <summary>
         /// Name of the label (required).
         /// </summary>
+        /// <remarks>
+        /// Emoji can be added to label names, using either native emoji or colon-style markup. For example,
+        /// typing :strawberry: will render the emoji for strawberry. For a full list of available emoji and codes, see http://emoji-cheat-sheet.com/.
+        /// </remarks>
         public string Name { get; set; }
 
         /// <summary>
         /// Color of the label (required).
         /// </summary>
+        /// <remarks>
+        /// The hexadecimal color code for the label, without the leading #.
+        /// </remarks>
         public string Color
         {
             get { return _color; }
@@ -49,6 +58,11 @@ public string Color
             }
         }
 
+        /// <summary>
+        /// A short description of the label (optional).
+        /// </summary>
+        public string Description { get; set; }
+
         internal string DebuggerDisplay
         {
             get
diff --git a/Octokit/Models/Response/Label.cs b/Octokit/Models/Response/Label.cs
index f962aefd34..e11b94d3bc 100644
--- a/Octokit/Models/Response/Label.cs
+++ b/Octokit/Models/Response/Label.cs
@@ -8,11 +8,13 @@ public class Label
     {
         public Label() { }
 
-        public Label(string url, string name, string color)
+        public Label(string url, string name, string color, string description, bool @default)
         {
             Url = url;
             Name = name;
             Color = color;
+            Description = description;
+            IsDefault = @default;
         }
 
         /// <summary>
@@ -30,6 +32,16 @@ public Label(string url, string name, string color)
         /// </summary>
         public string Color { get; protected set; }
 
+        /// <summary>
+        /// Description of the label
+        /// </summary>
+        public string Description { get; protected set; }
+
+        /// <summary>
+        /// Is default label
+        /// </summary>
+        public bool IsDefault { get; protected set; }
+
         internal string DebuggerDisplay
         {
             get { return string.Format(CultureInfo.InvariantCulture, "Name: {0} Color: {1}", Name, Color); }

From 646aeafd03960a70e809a7270978e5bc48f1d5c7 Mon Sep 17 00:00:00 2001
From: Jozef Izso <jozef.izso@gmail.com>
Date: Sun, 13 May 2018 11:16:23 +0200
Subject: [PATCH 02/10] Include correct API header in all Labels calls

---
 .../Clients/IssuesLabelsClientTests.cs        |  4 +--
 .../Clients/IssuesLabelsClientTests.cs        | 28 +++++++++----------
 Octokit/Clients/IssuesLabelsClient.cs         | 16 +++++------
 Octokit/Models/Request/LabelUpdate.cs         |  4 +--
 Octokit/Models/Request/NewLabel.cs            |  4 +--
 Octokit/Models/Response/Label.cs              |  4 +--
 6 files changed, 28 insertions(+), 32 deletions(-)

diff --git a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
index c6d148c4fc..4cc6a6df38 100644
--- a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
@@ -29,7 +29,7 @@ public IssuesLabelsClientTests()
     public async Task CanListIssueLabelsForAnIssue()
     {
         var newIssue = new NewIssue("A test issue") { Body = "A new unassigned issue" };
-        var newLabel = new NewLabel("test label", "FFFFFF", "Test label description.");
+        var newLabel = new NewLabel("test label", "FFFFFF");
 
         var label = await _issuesLabelsClient.Create(_context.RepositoryOwner, _context.RepositoryName, newLabel);
         var issue = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue);
@@ -53,7 +53,7 @@ public async Task CanListIssueLabelsForAnIssue()
     public async Task CanListIssueLabelsForAnIssueWithRepositoryId()
     {
         var newIssue = new NewIssue("A test issue") { Body = "A new unassigned issue" };
-        var newLabel = new NewLabel("test label", "FFFFFF", "Test label description.");
+        var newLabel = new NewLabel("test label", "FFFFFF");
 
         var label = await _issuesLabelsClient.Create(_context.Repository.Id, newLabel);
         var issue = await _issuesClient.Create(_context.RepositoryOwner, _context.RepositoryName, newIssue);
diff --git a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
index b651723cad..63a8dca424 100644
--- a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
@@ -28,7 +28,7 @@ public async Task RequestsCorrectUrl()
 
                 await client.GetAllForIssue("fake", "repo", 42);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -39,7 +39,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 await client.GetAllForIssue(1, 42);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -57,7 +57,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 await client.GetAllForIssue("fake", "repo", 42, options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -75,7 +75,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 await client.GetAllForIssue(1, 42, options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -108,7 +108,7 @@ public async Task RequestsCorrectUrl()
 
                 await client.GetAllForRepository("fake", "repo");
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -119,7 +119,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 await client.GetAllForRepository(1);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -137,7 +137,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 await client.GetAllForRepository("fake", "repo", options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -155,7 +155,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 await client.GetAllForRepository(1, options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -188,7 +188,7 @@ public async Task RequestsCorrectUrl()
 
                 await client.GetAllForMilestone("fake", "repo", 42);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -199,7 +199,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 await client.GetAllForMilestone(1, 42);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), Args.ApiOptions);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), null, "application/vnd.github.symmetra-preview+json", Args.ApiOptions);
             }
 
             [Fact]
@@ -217,7 +217,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 await client.GetAllForMilestone("fake", "repo", 42, options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -235,7 +235,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 await client.GetAllForMilestone(1, 42, options);
 
-                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), options);
+                connection.Received().GetAll<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), null, "application/vnd.github.symmetra-preview+json", options);
             }
 
             [Fact]
@@ -268,7 +268,7 @@ public async Task RequestsCorrectUrl()
 
                 await client.Get("fake", "repo", "label");
 
-                connection.Received().Get<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels/label"));
+                connection.Received().Get<Label>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels/label"), null, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -279,7 +279,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 await client.Get(1, "label");
 
-                connection.Received().Get<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels/label"));
+                connection.Received().Get<Label>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels/label"), null, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
diff --git a/Octokit/Clients/IssuesLabelsClient.cs b/Octokit/Clients/IssuesLabelsClient.cs
index 86a1a94f17..f929ff2508 100644
--- a/Octokit/Clients/IssuesLabelsClient.cs
+++ b/Octokit/Clients/IssuesLabelsClient.cs
@@ -62,7 +62,7 @@ public Task<IReadOnlyList<Label>> GetAllForIssue(string owner, string name, int
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.IssueLabels(owner, name, number), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.IssueLabels(owner, name, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -78,7 +78,7 @@ public Task<IReadOnlyList<Label>> GetAllForIssue(long repositoryId, int number,
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.IssueLabels(repositoryId, number), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.IssueLabels(repositoryId, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -124,7 +124,7 @@ public Task<IReadOnlyList<Label>> GetAllForRepository(string owner, string name,
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.Labels(owner, name), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.Labels(owner, name), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -139,7 +139,7 @@ public Task<IReadOnlyList<Label>> GetAllForRepository(long repositoryId, ApiOpti
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.Labels(repositoryId), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.Labels(repositoryId), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -188,7 +188,7 @@ public Task<IReadOnlyList<Label>> GetAllForMilestone(string owner, string name,
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.MilestoneLabels(owner, name, number), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.MilestoneLabels(owner, name, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -204,7 +204,7 @@ public Task<IReadOnlyList<Label>> GetAllForMilestone(long repositoryId, int numb
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return ApiConnection.GetAll<Label>(ApiUrls.MilestoneLabels(repositoryId, number), options);
+            return ApiConnection.GetAll<Label>(ApiUrls.MilestoneLabels(repositoryId, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -222,7 +222,7 @@ public Task<Label> Get(string owner, string name, string labelName)
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return ApiConnection.Get<Label>(ApiUrls.Label(owner, name, labelName));
+            return ApiConnection.Get<Label>(ApiUrls.Label(owner, name, labelName), null, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -237,7 +237,7 @@ public Task<Label> Get(long repositoryId, string labelName)
         {
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return ApiConnection.Get<Label>(ApiUrls.Label(repositoryId, labelName));
+            return ApiConnection.Get<Label>(ApiUrls.Label(repositoryId, labelName), null, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
diff --git a/Octokit/Models/Request/LabelUpdate.cs b/Octokit/Models/Request/LabelUpdate.cs
index b9f72afea0..db576c1873 100644
--- a/Octokit/Models/Request/LabelUpdate.cs
+++ b/Octokit/Models/Request/LabelUpdate.cs
@@ -18,15 +18,13 @@ public class LabelUpdate
         /// </summary>
         /// <param name="name">The name of the label.</param>
         /// <param name="color">The color of the label.</param>
-        /// <param name="description">The description of the label.</param>
-        public LabelUpdate(string name, string color, string description = null)
+        public LabelUpdate(string name, string color)
         {
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(color, nameof(color));
 
             Name = name;
             Color = color;
-            Description = description;
         }
 
         /// <summary>
diff --git a/Octokit/Models/Request/NewLabel.cs b/Octokit/Models/Request/NewLabel.cs
index 83cdbae660..30fee49092 100644
--- a/Octokit/Models/Request/NewLabel.cs
+++ b/Octokit/Models/Request/NewLabel.cs
@@ -18,15 +18,13 @@ public class NewLabel
         /// </summary>
         /// <param name="name">The name of the label.</param>
         /// <param name="color">The color of the label.</param>
-        /// <param name="description">The description of the label.</param>
-        public NewLabel(string name, string color, string description = null)
+        public NewLabel(string name, string color)
         {
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(color, nameof(color));
 
             Name = name;
             Color = color;
-            Description = description;
         }
 
         /// <summary>
diff --git a/Octokit/Models/Response/Label.cs b/Octokit/Models/Response/Label.cs
index e11b94d3bc..e387963c34 100644
--- a/Octokit/Models/Response/Label.cs
+++ b/Octokit/Models/Response/Label.cs
@@ -14,7 +14,7 @@ public Label(string url, string name, string color, string description, bool @de
             Name = name;
             Color = color;
             Description = description;
-            IsDefault = @default;
+            Default = @default;
         }
 
         /// <summary>
@@ -40,7 +40,7 @@ public Label(string url, string name, string color, string description, bool @de
         /// <summary>
         /// Is default label
         /// </summary>
-        public bool IsDefault { get; protected set; }
+        public bool Default { get; protected set; }
 
         internal string DebuggerDisplay
         {

From e1de738487ce1d7952040d215cfcebab9c85a747 Mon Sep 17 00:00:00 2001
From: Jozef Izso <jozef.izso@gmail.com>
Date: Sun, 13 May 2018 12:20:00 +0200
Subject: [PATCH 03/10] Add integration tests for Create and Update methods for
 labels

---
 .../Clients/IssuesLabelsClientTests.cs        | 46 +++++++++++++++++++
 1 file changed, 46 insertions(+)

diff --git a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
index 4cc6a6df38..e95b7c7f80 100644
--- a/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests.Integration/Clients/IssuesLabelsClientTests.cs
@@ -794,6 +794,7 @@ public async Task CanRetrieveIssueLabelByName()
 
         Assert.Equal(label.Name, issueLabelLookupByName.Name);
         Assert.Equal(label.Color, issueLabelLookupByName.Color);
+        Assert.Equal(label.Description, issueLabelLookupByName.Description);
     }
 
     [IntegrationTest]
@@ -809,6 +810,51 @@ public async Task CanRetrieveIssueLabelByNameWithRepositoryId()
         Assert.Equal(label.Color, issueLabelLookupByName.Color);
     }
 
+    [IntegrationTest]
+    public async Task CanCreateIssueLabel()
+    {
+        var newLabel = new NewLabel("test label", "FFFFFF");
+        var label = await _issuesLabelsClient.Create(_context.RepositoryOwner, _context.RepositoryName, newLabel);
+        Assert.NotNull(label);
+
+        var issueLabelLookupByName = await _issuesLabelsClient.Get(_context.RepositoryOwner, _context.RepositoryName, label.Name);
+
+        Assert.Equal(label.Name, issueLabelLookupByName.Name);
+        Assert.Equal(label.Color, issueLabelLookupByName.Color);
+    }
+
+    [IntegrationTest]
+    public async Task CanCreateIssueLabelWithDescription()
+    {
+        var newLabel = new NewLabel("test label", "FFFFFF") { Description = "Test label description." };
+        var label = await _issuesLabelsClient.Create(_context.RepositoryOwner, _context.RepositoryName, newLabel);
+        Assert.NotNull(label);
+
+        var issueLabelLookupByName = await _issuesLabelsClient.Get(_context.RepositoryOwner, _context.RepositoryName, label.Name);
+
+        Assert.Equal(label.Name, issueLabelLookupByName.Name);
+        Assert.Equal(label.Color, issueLabelLookupByName.Color);
+        Assert.Equal(label.Description, issueLabelLookupByName.Description);
+    }
+
+    [IntegrationTest]
+    public async Task CanUpdateIssueLabel()
+    {
+        var newLabel = new NewLabel("test label", "FFFFFF") { Description = "Test label description." };
+        var label = await _issuesLabelsClient.Create(_context.RepositoryOwner, _context.RepositoryName, newLabel);
+        Assert.NotNull(label);
+
+        var labelUpdate = new LabelUpdate("test label", "000000") { Description = "Updated label description." };
+        label = await _issuesLabelsClient.Update(_context.RepositoryOwner, _context.RepositoryName, labelUpdate.Name, labelUpdate);
+        Assert.NotNull(label);
+
+        var issueLabelLookupByName = await _issuesLabelsClient.Get(_context.RepositoryOwner, _context.RepositoryName, label.Name);
+
+        Assert.Equal(labelUpdate.Name, issueLabelLookupByName.Name);
+        Assert.Equal(labelUpdate.Color, issueLabelLookupByName.Color);
+        Assert.Equal(labelUpdate.Description, issueLabelLookupByName.Description);
+    }
+
     [IntegrationTest]
     public async Task CanDeleteIssueLabelByName()
     {

From 13782f2feb5a06887dc3caf2780c3c5cf715c943 Mon Sep 17 00:00:00 2001
From: Jozef Izso <jozef.izso@gmail.com>
Date: Sun, 13 May 2018 21:19:48 +0200
Subject: [PATCH 04/10] Use improved labels API in observable client

---
 .../Clients/ObservableIssuesLabelsClient.cs   | 12 +++++-----
 .../ObservableIssuesLabelsClientTests.cs      | 24 +++++++++----------
 2 files changed, 18 insertions(+), 18 deletions(-)

diff --git a/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs b/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
index ff23308b4c..8a9753162d 100644
--- a/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
+++ b/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
@@ -71,7 +71,7 @@ public IObservable<Label> GetAllForIssue(string owner, string name, int number,
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.IssueLabels(owner, name, number), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.IssueLabels(owner, name, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -87,7 +87,7 @@ public IObservable<Label> GetAllForIssue(long repositoryId, int number, ApiOptio
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.IssueLabels(repositoryId, number), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.IssueLabels(repositoryId, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -133,7 +133,7 @@ public IObservable<Label> GetAllForRepository(string owner, string name, ApiOpti
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.Labels(owner, name), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.Labels(owner, name), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -148,7 +148,7 @@ public IObservable<Label> GetAllForRepository(long repositoryId, ApiOptions opti
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.Labels(repositoryId), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.Labels(repositoryId), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -197,7 +197,7 @@ public IObservable<Label> GetAllForMilestone(string owner, string name, int numb
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.MilestoneLabels(owner, name, number), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.MilestoneLabels(owner, name, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
@@ -213,7 +213,7 @@ public IObservable<Label> GetAllForMilestone(long repositoryId, int number, ApiO
         {
             Ensure.ArgumentNotNull(options, nameof(options));
 
-            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.MilestoneLabels(repositoryId, number), options);
+            return _connection.GetAndFlattenAllPages<Label>(ApiUrls.MilestoneLabels(repositoryId, number), null, AcceptHeaders.LabelsApiPreview, options);
         }
 
         /// <summary>
diff --git a/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs b/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
index fe63561c1f..839592652a 100644
--- a/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
@@ -30,7 +30,7 @@ public async Task RequestsCorrectUrl()
 
                 client.GetAllForIssue("fake", "repo", 42);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -42,7 +42,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 client.GetAllForIssue(1, 42);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -61,7 +61,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 client.GetAllForIssue("fake", "repo", 42, options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -80,7 +80,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 client.GetAllForIssue(1, 42, options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -114,7 +114,7 @@ public async Task RequestsCorrectUrl()
 
                 client.GetAllForRepository("fake", "repo");
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -126,7 +126,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 client.GetAllForRepository(1);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -145,7 +145,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 client.GetAllForRepository("fake", "repo", options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -164,7 +164,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 client.GetAllForRepository(1, options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -198,7 +198,7 @@ public async Task RequestsCorrectUrl()
 
                 client.GetAllForMilestone("fake", "repo", 42);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -210,7 +210,7 @@ public async Task RequestsCorrectUrlWithRepositoryId()
 
                 client.GetAllForMilestone(1, 42);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), Args.EmptyDictionary, null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), Args.EmptyDictionary, "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -229,7 +229,7 @@ public async Task RequestsCorrectUrlWithApiOptions()
 
                 client.GetAllForMilestone("fake", "repo", 42, options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/milestones/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -248,7 +248,7 @@ public async Task RequestsCorrectUrlWithApiOptionsWithRepositoryId()
 
                 client.GetAllForMilestone(1, 42, options);
 
-                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), null);
+                connection.Received().Get<List<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/milestones/42/labels"), Arg.Is<Dictionary<string, string>>(d => d.Count == 2), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]

From 28272867b37d8f648fba627a7711c38d3a93f8a2 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 14 May 2018 22:03:40 +1000
Subject: [PATCH 05/10] found even more endpoints that need the preview header!

---
 Octokit.Tests/Clients/IssuesLabelsClientTests.cs | 8 ++++----
 Octokit/Clients/IssuesLabelsClient.cs            | 8 ++++----
 2 files changed, 8 insertions(+), 8 deletions(-)

diff --git a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
index 63a8dca424..de91df21d5 100644
--- a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
@@ -313,7 +313,7 @@ public void PostsToCorrectUrl()
 
                 client.AddToIssue("fake", "repo", 42, labels);
 
-                connection.Received().Post<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Any<string[]>());
+                connection.Received().Post<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Any<string[]>(), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -324,7 +324,7 @@ public void PostsToCorrectUrlWithRepositoryId()
 
                 client.AddToIssue(1, 42, labels);
 
-                connection.Received().Post<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Any<string[]>());
+                connection.Received().Post<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Any<string[]>(), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -398,7 +398,7 @@ public void PutsToCorrectUrl()
 
                 client.ReplaceAllForIssue("fake", "repo", 42, labels);
 
-                connection.Received().Put<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Any<string[]>());
+                connection.Received().Put<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels"), Arg.Any<string[]>(), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -409,7 +409,7 @@ public void PutsToCorrectUrlWithRepositoryId()
 
                 client.ReplaceAllForIssue(1, 42, labels);
 
-                connection.Received().Put<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Any<string[]>());
+                connection.Received().Put<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels"), Arg.Any<string[]>(), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
diff --git a/Octokit/Clients/IssuesLabelsClient.cs b/Octokit/Clients/IssuesLabelsClient.cs
index f929ff2508..9c545cec50 100644
--- a/Octokit/Clients/IssuesLabelsClient.cs
+++ b/Octokit/Clients/IssuesLabelsClient.cs
@@ -359,7 +359,7 @@ public Task<IReadOnlyList<Label>> AddToIssue(string owner, string name, int numb
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(labels, nameof(labels));
 
-            return ApiConnection.Post<IReadOnlyList<Label>>(ApiUrls.IssueLabels(owner, name, number), labels);
+            return ApiConnection.Post<IReadOnlyList<Label>>(ApiUrls.IssueLabels(owner, name, number), labels, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -375,7 +375,7 @@ public Task<IReadOnlyList<Label>> AddToIssue(long repositoryId, int number, stri
         {
             Ensure.ArgumentNotNull(labels, nameof(labels));
 
-            return ApiConnection.Post<IReadOnlyList<Label>>(ApiUrls.IssueLabels(repositoryId, number), labels);
+            return ApiConnection.Post<IReadOnlyList<Label>>(ApiUrls.IssueLabels(repositoryId, number), labels, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -429,7 +429,7 @@ public Task<IReadOnlyList<Label>> ReplaceAllForIssue(string owner, string name,
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNull(labels, nameof(labels));
 
-            return ApiConnection.Put<IReadOnlyList<Label>>(ApiUrls.IssueLabels(owner, name, number), labels);
+            return ApiConnection.Put<IReadOnlyList<Label>>(ApiUrls.IssueLabels(owner, name, number), labels, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -445,7 +445,7 @@ public Task<IReadOnlyList<Label>> ReplaceAllForIssue(long repositoryId, int numb
         {
             Ensure.ArgumentNotNull(labels, nameof(labels));
 
-            return ApiConnection.Put<IReadOnlyList<Label>>(ApiUrls.IssueLabels(repositoryId, number), labels);
+            return ApiConnection.Put<IReadOnlyList<Label>>(ApiUrls.IssueLabels(repositoryId, number), labels, AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>

From 9f46ca129184b128c1e7644e7842edaf99e94421 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 14 May 2018 22:11:15 +1000
Subject: [PATCH 06/10] RemoveFromIssue actually returns the list of remaining
 labels rather than null.  This change should be source compatible but not
 binary compatible

---
 .../Clients/IObservableIssuesLabelsClient.cs         |  4 ++--
 .../Clients/ObservableIssuesLabelsClient.cs          | 12 ++++++++----
 Octokit.Tests/Clients/IssuesLabelsClientTests.cs     |  4 ++--
 .../Reactive/ObservableIssuesLabelsClientTests.cs    |  4 ++--
 Octokit/Clients/IIssuesLabelsClient.cs               |  4 ++--
 Octokit/Clients/IssuesLabelsClient.cs                |  8 ++++----
 6 files changed, 20 insertions(+), 16 deletions(-)

diff --git a/Octokit.Reactive/Clients/IObservableIssuesLabelsClient.cs b/Octokit.Reactive/Clients/IObservableIssuesLabelsClient.cs
index 4fe6f11d73..aa84153ee5 100644
--- a/Octokit.Reactive/Clients/IObservableIssuesLabelsClient.cs
+++ b/Octokit.Reactive/Clients/IObservableIssuesLabelsClient.cs
@@ -263,7 +263,7 @@ public interface IObservableIssuesLabelsClient
         /// <param name="name">The name of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        IObservable<Unit> RemoveFromIssue(string owner, string name, int number, string labelName);
+        IObservable<Label> RemoveFromIssue(string owner, string name, int number, string labelName);
 
         /// <summary>
         /// Removes a label from an issue
@@ -274,7 +274,7 @@ public interface IObservableIssuesLabelsClient
         /// <param name="repositoryId">The Id of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        IObservable<Unit> RemoveFromIssue(long repositoryId, int number, string labelName);
+        IObservable<Label> RemoveFromIssue(long repositoryId, int number, string labelName);
 
         /// <summary>
         /// Replaces all labels on the specified issues with the provided labels
diff --git a/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs b/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
index 8a9753162d..f3400c140d 100644
--- a/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
+++ b/Octokit.Reactive/Clients/ObservableIssuesLabelsClient.cs
@@ -401,13 +401,15 @@ public IObservable<Label> AddToIssue(long repositoryId, int number, string[] lab
         /// <param name="name">The name of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        public IObservable<Unit> RemoveFromIssue(string owner, string name, int number, string labelName)
+        public IObservable<Label> RemoveFromIssue(string owner, string name, int number, string labelName)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, nameof(owner));
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return _client.RemoveFromIssue(owner, name, number, labelName).ToObservable();
+            return _client.RemoveFromIssue(owner, name, number, labelName)
+                .ToObservable()
+                .SelectMany(x => x); // HACK: DELETE is not compatible with GetAndFlattenPages
         }
 
         /// <summary>
@@ -419,11 +421,13 @@ public IObservable<Unit> RemoveFromIssue(string owner, string name, int number,
         /// <param name="repositoryId">The Id of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        public IObservable<Unit> RemoveFromIssue(long repositoryId, int number, string labelName)
+        public IObservable<Label> RemoveFromIssue(long repositoryId, int number, string labelName)
         {
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return _client.RemoveFromIssue(repositoryId, number, labelName).ToObservable();
+            return _client.RemoveFromIssue(repositoryId, number, labelName)
+                .ToObservable()
+                .SelectMany(x => x); // HACK: DELETE is not compatible with GetAndFlattenPages
         }
 
         /// <summary>
diff --git a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
index de91df21d5..dfe82b7963 100644
--- a/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Clients/IssuesLabelsClientTests.cs
@@ -353,7 +353,7 @@ public void DeleteCorrectUrl()
 
                 client.RemoveFromIssue("fake", "repo", 42, "label");
 
-                connection.Received().Delete(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels/label"));
+                connection.Received().Delete<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels/label"), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -364,7 +364,7 @@ public void DeleteCorrectUrlWithRepositoryId()
 
                 client.RemoveFromIssue(1, 42, "label");
 
-                connection.Received().Delete(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels/label"));
+                connection.Received().Delete<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels/label"), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
diff --git a/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs b/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
index 839592652a..57b030548a 100644
--- a/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
+++ b/Octokit.Tests/Reactive/ObservableIssuesLabelsClientTests.cs
@@ -367,7 +367,7 @@ public void DeleteCorrectUrl()
 
                 client.RemoveFromIssue("fake", "repo", 42, "label");
 
-                connection.Received().Delete(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels/label"));
+                connection.Received().Delete<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repos/fake/repo/issues/42/labels/label"), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
@@ -379,7 +379,7 @@ public void DeleteCorrectUrlWithRepositoryId()
 
                 client.RemoveFromIssue(1, 42, "label");
 
-                connection.Received().Delete(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels/label"));
+                connection.Received().Delete<IReadOnlyList<Label>>(Arg.Is<Uri>(u => u.ToString() == "repositories/1/issues/42/labels/label"), "application/vnd.github.symmetra-preview+json");
             }
 
             [Fact]
diff --git a/Octokit/Clients/IIssuesLabelsClient.cs b/Octokit/Clients/IIssuesLabelsClient.cs
index e34d8304ad..2c535ebec1 100644
--- a/Octokit/Clients/IIssuesLabelsClient.cs
+++ b/Octokit/Clients/IIssuesLabelsClient.cs
@@ -263,7 +263,7 @@ public interface IIssuesLabelsClient
         /// <param name="name">The name of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        Task RemoveFromIssue(string owner, string name, int number, string labelName);
+        Task<IReadOnlyList<Label>> RemoveFromIssue(string owner, string name, int number, string labelName);
 
         /// <summary>
         /// Removes a label from an issue
@@ -274,7 +274,7 @@ public interface IIssuesLabelsClient
         /// <param name="repositoryId">The Id of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        Task RemoveFromIssue(long repositoryId, int number, string labelName);
+        Task<IReadOnlyList<Label>> RemoveFromIssue(long repositoryId, int number, string labelName);
 
         /// <summary>
         /// Replaces all labels on the specified issues with the provided labels
diff --git a/Octokit/Clients/IssuesLabelsClient.cs b/Octokit/Clients/IssuesLabelsClient.cs
index 9c545cec50..15a51be6c7 100644
--- a/Octokit/Clients/IssuesLabelsClient.cs
+++ b/Octokit/Clients/IssuesLabelsClient.cs
@@ -388,13 +388,13 @@ public Task<IReadOnlyList<Label>> AddToIssue(long repositoryId, int number, stri
         /// <param name="name">The name of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        public Task RemoveFromIssue(string owner, string name, int number, string labelName)
+        public Task<IReadOnlyList<Label>> RemoveFromIssue(string owner, string name, int number, string labelName)
         {
             Ensure.ArgumentNotNullOrEmptyString(owner, nameof(owner));
             Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return ApiConnection.Delete(ApiUrls.IssueLabel(owner, name, number, labelName));
+            return ApiConnection.Delete<IReadOnlyList<Label>>(ApiUrls.IssueLabel(owner, name, number, labelName), AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>
@@ -406,11 +406,11 @@ public Task RemoveFromIssue(string owner, string name, int number, string labelN
         /// <param name="repositoryId">The Id of the repository</param>
         /// <param name="number">The number of the issue</param>
         /// <param name="labelName">The name of the label to remove</param>
-        public Task RemoveFromIssue(long repositoryId, int number, string labelName)
+        public Task<IReadOnlyList<Label>> RemoveFromIssue(long repositoryId, int number, string labelName)
         {
             Ensure.ArgumentNotNullOrEmptyString(labelName, nameof(labelName));
 
-            return ApiConnection.Delete(ApiUrls.IssueLabel(repositoryId, number, labelName));
+            return ApiConnection.Delete<IReadOnlyList<Label>>(ApiUrls.IssueLabel(repositoryId, number, labelName), AcceptHeaders.LabelsApiPreview);
         }
 
         /// <summary>

From 1472fd3e83b9ac573941ba31210c5192bef9ffcf Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 14 May 2018 23:39:07 +1000
Subject: [PATCH 07/10] Implement new labels search method in SearchClient

---
 .../Clients/SearchClientTests.cs              |  10 ++
 Octokit.Tests/Clients/SearchClientTests.cs    | 106 ++++++++++++++++++
 Octokit/Clients/ISearchClient.cs              |   7 ++
 Octokit/Clients/SearchClient.cs               |   6 +
 Octokit/Helpers/ApiUrls.cs                    |   9 ++
 Octokit/Models/Request/BaseSearchRequest.cs   |  17 ++-
 Octokit/Models/Request/SearchLabelsRequest.cs |  88 +++++++++++++++
 Octokit/Models/Response/SearchLabelsResult.cs |  17 +++
 8 files changed, 259 insertions(+), 1 deletion(-)
 create mode 100644 Octokit/Models/Request/SearchLabelsRequest.cs
 create mode 100644 Octokit/Models/Response/SearchLabelsResult.cs

diff --git a/Octokit.Tests.Integration/Clients/SearchClientTests.cs b/Octokit.Tests.Integration/Clients/SearchClientTests.cs
index 7f27b4d636..303fdfe0be 100644
--- a/Octokit.Tests.Integration/Clients/SearchClientTests.cs
+++ b/Octokit.Tests.Integration/Clients/SearchClientTests.cs
@@ -616,4 +616,14 @@ public async Task SearchForExcludedBase()
         // Ensure no items from the first search are in the results for the second
         Assert.DoesNotContain(issues.Items, x1 => otherIssues.Items.Any(x2 => x2.Id == x1.Id));
     }
+
+    [IntegrationTest]
+    public async Task SearchForAllLabelsUsingTermAndRepositoryId()
+    {
+        var request = new SearchLabelsRequest("category", 7528679);
+        
+        var labels = await _gitHubClient.Search.SearchLabels(request);
+
+        Assert.NotEmpty(labels.Items);
+    }
 }
diff --git a/Octokit.Tests/Clients/SearchClientTests.cs b/Octokit.Tests/Clients/SearchClientTests.cs
index a4eba60b66..d90f0e3ec6 100644
--- a/Octokit.Tests/Clients/SearchClientTests.cs
+++ b/Octokit.Tests/Clients/SearchClientTests.cs
@@ -1675,5 +1675,111 @@ await Assert.ThrowsAsync<RepositoryFormatException>(
                     async () => await client.SearchCode(request));
             }
         }
+
+        public class TheSearchLabelsMethod
+        {
+            [Fact]
+            public void RequestsTheCorrectUrl()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                client.SearchLabels(new SearchLabelsRequest("something", 1));
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Any<Dictionary<string, string>>(),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+
+            [Fact]
+            public async Task EnsuresNonNullArguments()
+            {
+                var client = new SearchClient(Substitute.For<IApiConnection>());
+                await Assert.ThrowsAsync<ArgumentNullException>(() => client.SearchLabels(null));
+            }
+
+            [Fact]
+            public void TestingTheTermParameter()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                var request = new SearchLabelsRequest("something", 1);
+
+                client.SearchLabels(request);
+
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Is<Dictionary<string, string>>(d => 
+                        d["q"] == "something" &&
+                        d["repository_id"] == "1"),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+
+            [Fact]
+            public void TestingTheSortParameter()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                var request = new SearchLabelsRequest("something", 1);
+                request.SortField = LabelSearchSort.Created;
+
+                client.SearchLabels(request);
+
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Is<Dictionary<string, string>>(d => d["sort"] == "created"),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+
+            [Fact]
+            public void TestingTheOrderParameter()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                var request = new SearchLabelsRequest("something", 1);
+                request.SortField = LabelSearchSort.Created;
+                request.Order = SortDirection.Ascending;
+
+                client.SearchLabels(request);
+
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Is<Dictionary<string, string>>(d =>
+                        d["sort"] == "created" &&
+                        d["order"] == "asc"),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+
+            [Fact]
+            public void TestingTheDefaultOrderParameter()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                var request = new SearchLabelsRequest("something", 1);
+
+                client.SearchLabels(request);
+
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Is<Dictionary<string, string>>(d => d["order"] == "desc"),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+
+            [Fact]
+            public void TestingTheRepositoryIdParameter()
+            {
+                var connection = Substitute.For<IApiConnection>();
+                var client = new SearchClient(connection);
+                var request = new SearchLabelsRequest("something", 1);
+
+                client.SearchLabels(request);
+
+                connection.Received().Get<SearchLabelsResult>(
+                    Arg.Is<Uri>(u => u.ToString() == "search/labels"),
+                    Arg.Is<Dictionary<string, string>>(d =>
+                        d["q"] == "something" && 
+                        d["repository_id"] == "1"),
+                    "application/vnd.github.symmetra-preview+json");
+            }
+        }
     }
 }
diff --git a/Octokit/Clients/ISearchClient.cs b/Octokit/Clients/ISearchClient.cs
index ed8b9f3ad9..c1c5ce6c6e 100644
--- a/Octokit/Clients/ISearchClient.cs
+++ b/Octokit/Clients/ISearchClient.cs
@@ -41,5 +41,12 @@ public interface ISearchClient
         /// <param name="search"></param>
         /// <returns>List of files</returns>
         Task<SearchCodeResult> SearchCode(SearchCodeRequest search);
+
+        /// <summary>
+        /// search labels
+        /// </summary>
+        /// <param name="search"></param>
+        /// <returns>List of labels</returns>
+        Task<SearchLabelsResult> SearchLabels(SearchLabelsRequest search);
     }
 }
\ No newline at end of file
diff --git a/Octokit/Clients/SearchClient.cs b/Octokit/Clients/SearchClient.cs
index aa6c802449..ee4589725b 100644
--- a/Octokit/Clients/SearchClient.cs
+++ b/Octokit/Clients/SearchClient.cs
@@ -66,5 +66,11 @@ public Task<SearchCodeResult> SearchCode(SearchCodeRequest search)
             Ensure.ArgumentNotNull(search, nameof(search));
             return ApiConnection.Get<SearchCodeResult>(ApiUrls.SearchCode(), search.Parameters);
         }
+
+        public Task<SearchLabelsResult> SearchLabels(SearchLabelsRequest search)
+        {
+            Ensure.ArgumentNotNull(search, nameof(search));
+            return ApiConnection.Get<SearchLabelsResult>(ApiUrls.SearchLabels(), search.Parameters, AcceptHeaders.LabelsApiPreview);
+        }
     }
 }
\ No newline at end of file
diff --git a/Octokit/Helpers/ApiUrls.cs b/Octokit/Helpers/ApiUrls.cs
index 53198747ad..cd978d596b 100644
--- a/Octokit/Helpers/ApiUrls.cs
+++ b/Octokit/Helpers/ApiUrls.cs
@@ -1781,6 +1781,15 @@ public static Uri SearchCode()
             return "search/code".FormatUri();
         }
 
+        /// <summary>
+        /// Creates the relative <see cref="Uri"/> for searching labels
+        /// </summary>
+        /// <returns></returns>
+        public static Uri SearchLabels()
+        {
+            return "search/labels".FormatUri();
+        }
+
         /// <summary>
         /// Returns the <see cref="Uri"/> for repository contributors.
         /// </summary>
diff --git a/Octokit/Models/Request/BaseSearchRequest.cs b/Octokit/Models/Request/BaseSearchRequest.cs
index 48ccf2586b..ce98a0d7a7 100644
--- a/Octokit/Models/Request/BaseSearchRequest.cs
+++ b/Octokit/Models/Request/BaseSearchRequest.cs
@@ -75,7 +75,10 @@ private string SortOrder
         /// <summary>
         /// All qualifiers that are used for this search
         /// </summary>
-        public abstract IReadOnlyList<string> MergedQualifiers();
+        public virtual IReadOnlyList<string> MergedQualifiers()
+        {
+            return new List<string>();
+        }
 
         /// <summary>
         /// Add qualifiers onto the search term
@@ -96,6 +99,14 @@ private string TermAndQualifiers
             }
         }
 
+        /// <summary>
+        /// Any additional parameters required by the derived class
+        /// </summary>
+        public virtual IDictionary<string, string> AdditionalParameters()
+        {
+            return new Dictionary<string, string>();
+        }
+
         /// <summary>
         /// Get the query parameters that will be appending onto the search
         /// </summary>
@@ -114,6 +125,10 @@ public IDictionary<string, string> Parameters
                 {
                     d.Add("sort", Sort);
                 }
+                foreach (var parameter in AdditionalParameters())
+                {
+                    d.Add(parameter.Key, parameter.Value);
+                }
                 return d;
             }
         }
diff --git a/Octokit/Models/Request/SearchLabelsRequest.cs b/Octokit/Models/Request/SearchLabelsRequest.cs
new file mode 100644
index 0000000000..c962cdde58
--- /dev/null
+++ b/Octokit/Models/Request/SearchLabelsRequest.cs
@@ -0,0 +1,88 @@
+using System.Collections.Generic;
+using System.Collections.ObjectModel;
+using System.Diagnostics;
+using System.Diagnostics.CodeAnalysis;
+using System.Globalization;
+using System.Linq;
+using Octokit.Internal;
+
+namespace Octokit
+{
+    /// <summary>
+    /// Searching Code/Files
+    /// http://developer.github.com/v3/search/#search-code
+    /// </summary>
+    [DebuggerDisplay("{DebuggerDisplay,nq}")]
+    public class SearchLabelsRequest : BaseSearchRequest
+    {
+        /// <summary>
+        /// Initializes a new instance of the <see cref="SearchLabelsRequest"/> class.
+        /// </summary>
+        public SearchLabelsRequest() : base()
+        {
+        }
+
+        /// <summary>
+        /// Initializes a new instance of the <see cref="SearchLabelsRequest"/> class.
+        /// </summary>
+        /// <param name="term">The search term.</param>
+        /// <param name="repositoryId">The repository to search in</param>
+        public SearchLabelsRequest(string term, long repositoryId) : base(term)
+        {
+            RepositoryId = repositoryId;
+        }
+
+        /// <summary>
+        /// Optional Sort field. Can only be indexed, which indicates how recently 
+        /// a file has been indexed by the GitHub search infrastructure. 
+        /// If not provided, results are sorted by best match.
+        /// </summary>
+        /// <remarks>
+        /// http://developer.github.com/v3/search/#search-code
+        /// </remarks>
+        public LabelSearchSort? SortField { get; set; }
+        public override string Sort
+        {
+            get { return SortField.ToParameter(); }
+        }
+
+        /// <summary>
+        /// Limits searches to a specific organization.
+        /// </summary>
+        /// <remarks>
+        /// https://help.github.com/articles/searching-code/#search-within-a-users-or-organizations-repositories
+        /// </remarks>
+        public long RepositoryId { get; set; }
+
+        public override IDictionary<string, string> AdditionalParameters()
+        {
+            return new Dictionary<string, string>
+            {
+                { "repository_id", RepositoryId.ToString() }
+            };
+        }
+
+        internal string DebuggerDisplay
+        {
+            get
+            {
+                return string.Format(CultureInfo.InvariantCulture, "Term: {0} Sort: {1} RepositoryId: {2}", Term, Sort, RepositoryId);
+            }
+        }
+    }
+
+    public enum LabelSearchSort
+    {
+        /// <summary>
+        /// search by created
+        /// </summary>
+        [Parameter(Value = "created")]
+        Created,
+        
+        /// <summary>
+        /// search by last updated
+        /// </summary>
+        [Parameter(Value = "updated")]
+        Updated
+    }
+}
diff --git a/Octokit/Models/Response/SearchLabelsResult.cs b/Octokit/Models/Response/SearchLabelsResult.cs
new file mode 100644
index 0000000000..aee442c59c
--- /dev/null
+++ b/Octokit/Models/Response/SearchLabelsResult.cs
@@ -0,0 +1,17 @@
+using System.Collections.Generic;
+using System.Diagnostics;
+using Octokit.Internal;
+
+namespace Octokit
+{
+    [DebuggerDisplay("{DebuggerDisplay,nq}")]
+    public class SearchLabelsResult : SearchResult<Label>
+    {
+        public SearchLabelsResult() { }
+
+        public SearchLabelsResult(int totalCount, bool incompleteResults, IReadOnlyList<Label> items)
+            : base(totalCount, incompleteResults, items)
+        {
+        }
+    }
+}
\ No newline at end of file

From 8fe42d33ccc7a30cceb00de3277b20a2d6c428df Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 14 May 2018 23:42:07 +1000
Subject: [PATCH 08/10] Implement reactive client SearchLabels

---
 Octokit.Reactive/Clients/IObservableSearchClient.cs |  7 +++++++
 Octokit.Reactive/Clients/ObservableSearchClient.cs  | 11 +++++++++++
 Octokit/Clients/SearchClient.cs                     |  5 +++++
 3 files changed, 23 insertions(+)

diff --git a/Octokit.Reactive/Clients/IObservableSearchClient.cs b/Octokit.Reactive/Clients/IObservableSearchClient.cs
index d3eb61358f..7cde5a1b77 100644
--- a/Octokit.Reactive/Clients/IObservableSearchClient.cs
+++ b/Octokit.Reactive/Clients/IObservableSearchClient.cs
@@ -38,5 +38,12 @@ public interface IObservableSearchClient
         /// <param name="search"></param>
         /// <returns>List of files</returns>
         IObservable<SearchCodeResult> SearchCode(SearchCodeRequest search);
+
+        /// <summary>
+        /// search labels
+        /// </summary>
+        /// <param name="search"></param>
+        /// <returns>List of labels</returns>
+        IObservable<SearchLabelsResult> SearchLabels(SearchLabelsRequest search);
     }
 }
\ No newline at end of file
diff --git a/Octokit.Reactive/Clients/ObservableSearchClient.cs b/Octokit.Reactive/Clients/ObservableSearchClient.cs
index 3505de266b..bae5affc52 100644
--- a/Octokit.Reactive/Clients/ObservableSearchClient.cs
+++ b/Octokit.Reactive/Clients/ObservableSearchClient.cs
@@ -64,5 +64,16 @@ public IObservable<SearchCodeResult> SearchCode(SearchCodeRequest search)
             Ensure.ArgumentNotNull(search, nameof(search));
             return _client.SearchCode(search).ToObservable();
         }
+
+        /// <summary>
+        /// search labels
+        /// </summary>
+        /// <param name="search"></param>
+        /// <returns>List of labels</returns>
+        public IObservable<SearchLabelsResult> SearchLabels(SearchLabelsRequest search)
+        {
+            Ensure.ArgumentNotNull(search, nameof(search));
+            return _client.SearchLabels(search).ToObservable();
+        }
     }
 }
\ No newline at end of file
diff --git a/Octokit/Clients/SearchClient.cs b/Octokit/Clients/SearchClient.cs
index ee4589725b..73eaeba341 100644
--- a/Octokit/Clients/SearchClient.cs
+++ b/Octokit/Clients/SearchClient.cs
@@ -67,6 +67,11 @@ public Task<SearchCodeResult> SearchCode(SearchCodeRequest search)
             return ApiConnection.Get<SearchCodeResult>(ApiUrls.SearchCode(), search.Parameters);
         }
 
+        /// <summary>
+        /// search labels
+        /// </summary>
+        /// <param name="search"></param>
+        /// <returns>List of labels</returns>
         public Task<SearchLabelsResult> SearchLabels(SearchLabelsRequest search)
         {
             Ensure.ArgumentNotNull(search, nameof(search));

From 7bce8aac43e7e152a3be6471ed8bc318bbf781e8 Mon Sep 17 00:00:00 2001
From: Jozef Izso <jozef.izso@gmail.com>
Date: Mon, 14 May 2018 19:28:00 +0200
Subject: [PATCH 09/10] Improve documentation for label search methods

---
 Octokit.Reactive/Clients/IObservableSearchClient.cs | 1 +
 Octokit.Reactive/Clients/ObservableSearchClient.cs  | 1 +
 Octokit/Clients/ISearchClient.cs                    | 1 +
 Octokit/Clients/SearchClient.cs                     | 1 +
 Octokit/Models/Request/SearchLabelsRequest.cs       | 5 +++--
 5 files changed, 7 insertions(+), 2 deletions(-)

diff --git a/Octokit.Reactive/Clients/IObservableSearchClient.cs b/Octokit.Reactive/Clients/IObservableSearchClient.cs
index 7cde5a1b77..4a47cd0540 100644
--- a/Octokit.Reactive/Clients/IObservableSearchClient.cs
+++ b/Octokit.Reactive/Clients/IObservableSearchClient.cs
@@ -41,6 +41,7 @@ public interface IObservableSearchClient
 
         /// <summary>
         /// search labels
+        /// https://developer.github.com/v3/search/#search-labels
         /// </summary>
         /// <param name="search"></param>
         /// <returns>List of labels</returns>
diff --git a/Octokit.Reactive/Clients/ObservableSearchClient.cs b/Octokit.Reactive/Clients/ObservableSearchClient.cs
index bae5affc52..9f9b2c7f32 100644
--- a/Octokit.Reactive/Clients/ObservableSearchClient.cs
+++ b/Octokit.Reactive/Clients/ObservableSearchClient.cs
@@ -67,6 +67,7 @@ public IObservable<SearchCodeResult> SearchCode(SearchCodeRequest search)
 
         /// <summary>
         /// search labels
+        /// https://developer.github.com/v3/search/#search-labels
         /// </summary>
         /// <param name="search"></param>
         /// <returns>List of labels</returns>
diff --git a/Octokit/Clients/ISearchClient.cs b/Octokit/Clients/ISearchClient.cs
index c1c5ce6c6e..2246b53eda 100644
--- a/Octokit/Clients/ISearchClient.cs
+++ b/Octokit/Clients/ISearchClient.cs
@@ -44,6 +44,7 @@ public interface ISearchClient
 
         /// <summary>
         /// search labels
+        /// https://developer.github.com/v3/search/#search-labels
         /// </summary>
         /// <param name="search"></param>
         /// <returns>List of labels</returns>
diff --git a/Octokit/Clients/SearchClient.cs b/Octokit/Clients/SearchClient.cs
index 73eaeba341..57731cc1aa 100644
--- a/Octokit/Clients/SearchClient.cs
+++ b/Octokit/Clients/SearchClient.cs
@@ -69,6 +69,7 @@ public Task<SearchCodeResult> SearchCode(SearchCodeRequest search)
 
         /// <summary>
         /// search labels
+        /// https://developer.github.com/v3/search/#search-labels
         /// </summary>
         /// <param name="search"></param>
         /// <returns>List of labels</returns>
diff --git a/Octokit/Models/Request/SearchLabelsRequest.cs b/Octokit/Models/Request/SearchLabelsRequest.cs
index c962cdde58..02fcafcee7 100644
--- a/Octokit/Models/Request/SearchLabelsRequest.cs
+++ b/Octokit/Models/Request/SearchLabelsRequest.cs
@@ -9,8 +9,8 @@
 namespace Octokit
 {
     /// <summary>
-    /// Searching Code/Files
-    /// http://developer.github.com/v3/search/#search-code
+    /// Search labels
+    /// https://developer.github.com/v3/search/#search-labels
     /// </summary>
     [DebuggerDisplay("{DebuggerDisplay,nq}")]
     public class SearchLabelsRequest : BaseSearchRequest
@@ -41,6 +41,7 @@ public SearchLabelsRequest(string term, long repositoryId) : base(term)
         /// http://developer.github.com/v3/search/#search-code
         /// </remarks>
         public LabelSearchSort? SortField { get; set; }
+
         public override string Sort
         {
             get { return SortField.ToParameter(); }

From ecbfd5bd6d3d161fa8f8bfca8a1d4b154719ad9c Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Thu, 17 May 2018 21:03:02 +1000
Subject: [PATCH 10/10] more comment tidy up

---
 Octokit/Models/Request/SearchLabelsRequest.cs | 11 ++---------
 1 file changed, 2 insertions(+), 9 deletions(-)

diff --git a/Octokit/Models/Request/SearchLabelsRequest.cs b/Octokit/Models/Request/SearchLabelsRequest.cs
index 02fcafcee7..dc067c9aa3 100644
--- a/Octokit/Models/Request/SearchLabelsRequest.cs
+++ b/Octokit/Models/Request/SearchLabelsRequest.cs
@@ -33,13 +33,9 @@ public SearchLabelsRequest(string term, long repositoryId) : base(term)
         }
 
         /// <summary>
-        /// Optional Sort field. Can only be indexed, which indicates how recently 
-        /// a file has been indexed by the GitHub search infrastructure. 
+        /// Optional Sort field. One of created or updated.
         /// If not provided, results are sorted by best match.
         /// </summary>
-        /// <remarks>
-        /// http://developer.github.com/v3/search/#search-code
-        /// </remarks>
         public LabelSearchSort? SortField { get; set; }
 
         public override string Sort
@@ -48,11 +44,8 @@ public override string Sort
         }
 
         /// <summary>
-        /// Limits searches to a specific organization.
+        /// The repository to search in
         /// </summary>
-        /// <remarks>
-        /// https://help.github.com/articles/searching-code/#search-within-a-users-or-organizations-repositories
-        /// </remarks>
         public long RepositoryId { get; set; }
 
         public override IDictionary<string, string> AdditionalParameters()