From c65de95d29e981b68555310e9ca63b983bbfbc75 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 09:48:44 -0700
Subject: [PATCH 01/22] add node_id to Deployments payloads (Deployment and
 DeploymentStatus and Account/User/Organization)

---
 Octokit/Models/Response/Account.cs          | 8 +++++++-
 Octokit/Models/Response/Deployment.cs       | 8 +++++++-
 Octokit/Models/Response/DeploymentStatus.cs | 8 +++++++-
 Octokit/Models/Response/Organization.cs     | 4 ++--
 Octokit/Models/Response/User.cs             | 4 ++--
 5 files changed, 25 insertions(+), 7 deletions(-)

diff --git a/Octokit/Models/Response/Account.cs b/Octokit/Models/Response/Account.cs
index 9538411933..9e242b9e9e 100644
--- a/Octokit/Models/Response/Account.cs
+++ b/Octokit/Models/Response/Account.cs
@@ -9,7 +9,7 @@ public abstract class Account
     {
         protected Account() { }
 
-        protected Account(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, AccountType type, string url)
+        protected Account(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, string nodeId, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, AccountType type, string url)
         {
             AvatarUrl = avatarUrl;
             Bio = bio;
@@ -28,6 +28,7 @@ protected Account(string avatarUrl, string bio, string blog, int collaborators,
             Location = location;
             Login = login;
             Name = name;
+            NodeId = nodeId;
             OwnedPrivateRepos = ownedPrivateRepos;
             Plan = plan;
             PrivateGists = privateGists;
@@ -103,6 +104,11 @@ protected Account(string avatarUrl, string bio, string blog, int collaborators,
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The account's geographic location.
         /// </summary>
diff --git a/Octokit/Models/Response/Deployment.cs b/Octokit/Models/Response/Deployment.cs
index a9ab8fe2cd..4d4e5be667 100644
--- a/Octokit/Models/Response/Deployment.cs
+++ b/Octokit/Models/Response/Deployment.cs
@@ -13,9 +13,10 @@ public class Deployment
     {
         public Deployment() { }
 
-        public Deployment(int id, string sha, string url, User creator, IReadOnlyDictionary<string, string> payload, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description, string statusesUrl, bool transientEnvironment, bool productionEnvironment)
+        public Deployment(int id, string nodeId, string sha, string url, User creator, IReadOnlyDictionary<string, string> payload, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description, string statusesUrl, bool transientEnvironment, bool productionEnvironment)
         {
             Id = id;
+            NodeId = nodeId;
             Sha = sha;
             Url = url;
             Creator = creator;
@@ -33,6 +34,11 @@ public Deployment(int id, string sha, string url, User creator, IReadOnlyDiction
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// 
         /// </summary>
diff --git a/Octokit/Models/Response/DeploymentStatus.cs b/Octokit/Models/Response/DeploymentStatus.cs
index 363144da5f..a7bef3ca05 100644
--- a/Octokit/Models/Response/DeploymentStatus.cs
+++ b/Octokit/Models/Response/DeploymentStatus.cs
@@ -11,9 +11,10 @@ public class DeploymentStatus
     {
         public DeploymentStatus() { }
 
-        public DeploymentStatus(int id, string url, DeploymentState state, User creator, IReadOnlyDictionary<string, string> payload, string targetUrl, string logUrl, string environmentUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description)
+        public DeploymentStatus(int id, string nodeId, string url, DeploymentState state, User creator, IReadOnlyDictionary<string, string> payload, string targetUrl, string logUrl, string environmentUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, string description)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             State = state;
             Creator = creator;
@@ -31,6 +32,11 @@ public DeploymentStatus(int id, string url, DeploymentState state, User creator,
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The API URL for this deployment status.
         /// </summary>
diff --git a/Octokit/Models/Response/Organization.cs b/Octokit/Models/Response/Organization.cs
index 9a1eea5620..9a0ad5dc25 100644
--- a/Octokit/Models/Response/Organization.cs
+++ b/Octokit/Models/Response/Organization.cs
@@ -10,8 +10,8 @@ public class Organization : Account
     {
         public Organization() { }
 
-        public Organization(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, string url, string billingAddress)
-            : base(avatarUrl, bio, blog, collaborators, company, createdAt, diskUsage, email, followers, following, hireable, htmlUrl, totalPrivateRepos, id, location, login, name, ownedPrivateRepos, plan, privateGists, publicGists, publicRepos, AccountType.Organization, url)
+        public Organization(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string nodeId, string location, string login, string name, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, string url, string billingAddress)
+            : base(avatarUrl, bio, blog, collaborators, company, createdAt, diskUsage, email, followers, following, hireable, htmlUrl, totalPrivateRepos, id, location, login, name, nodeId, ownedPrivateRepos, plan, privateGists, publicGists, publicRepos, AccountType.Organization, url)
         {
             BillingAddress = billingAddress;
         }
diff --git a/Octokit/Models/Response/User.cs b/Octokit/Models/Response/User.cs
index 3fcf93ec61..96a2c611a1 100644
--- a/Octokit/Models/Response/User.cs
+++ b/Octokit/Models/Response/User.cs
@@ -14,8 +14,8 @@ public class User : Account
     {
         public User() { }
 
-        public User(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, DateTimeOffset updatedAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, string url, RepositoryPermissions permissions, bool siteAdmin, string ldapDistinguishedName, DateTimeOffset? suspendedAt)
-            : base(avatarUrl, bio, blog, collaborators, company, createdAt, diskUsage, email, followers, following, hireable, htmlUrl, totalPrivateRepos, id, location, login, name, ownedPrivateRepos, plan, privateGists, publicGists, publicRepos, AccountType.User, url)
+        public User(string avatarUrl, string bio, string blog, int collaborators, string company, DateTimeOffset createdAt, DateTimeOffset updatedAt, int diskUsage, string email, int followers, int following, bool? hireable, string htmlUrl, int totalPrivateRepos, int id, string location, string login, string name, string nodeId, int ownedPrivateRepos, Plan plan, int privateGists, int publicGists, int publicRepos, string url, RepositoryPermissions permissions, bool siteAdmin, string ldapDistinguishedName, DateTimeOffset? suspendedAt)
+            : base(avatarUrl, bio, blog, collaborators, company, createdAt, diskUsage, email, followers, following, hireable, htmlUrl, totalPrivateRepos, id, location, login, name, nodeId, ownedPrivateRepos, plan, privateGists, publicGists, publicRepos, AccountType.User, url)
         {
             Permissions = permissions;
             SiteAdmin = siteAdmin;

From bee23f3baa3cae3eb3e4309cfa3b9a2be264277f Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 10:06:14 -0700
Subject: [PATCH 02/22] add node_id to gist responses

---
 Octokit/Models/Response/Gist.cs        | 8 +++++++-
 Octokit/Models/Response/GistComment.cs | 8 +++++++-
 Octokit/Models/Response/GistFork.cs    | 8 +++++++-
 3 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/Octokit/Models/Response/Gist.cs b/Octokit/Models/Response/Gist.cs
index 1c50a50c00..2ec6ffdc52 100644
--- a/Octokit/Models/Response/Gist.cs
+++ b/Octokit/Models/Response/Gist.cs
@@ -10,10 +10,11 @@ public class Gist
     {
         public Gist() { }
 
-        public Gist(string url, string id, string description, bool @public, User owner, IReadOnlyDictionary<string, GistFile> files, int comments, string commentsUrl, string htmlUrl, string gitPullUrl, string gitPushUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, IReadOnlyList<GistFork> forks, IReadOnlyList<GistHistory> history)
+        public Gist(string url, string id, string nodeId, string description, bool @public, User owner, IReadOnlyDictionary<string, GistFile> files, int comments, string commentsUrl, string htmlUrl, string gitPullUrl, string gitPushUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt, IReadOnlyList<GistFork> forks, IReadOnlyList<GistHistory> history)
         {
             Url = url;
             Id = id;
+            NodeId = nodeId;
             Description = description;
             Public = @public;
             Owner = owner;
@@ -42,6 +43,11 @@ public Gist(string url, string id, string description, bool @public, User owner,
         /// </remarks>
         public string Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// A description of the <see cref="Gist"/>.
         /// </summary>
diff --git a/Octokit/Models/Response/GistComment.cs b/Octokit/Models/Response/GistComment.cs
index e88d46ccba..9a9b773c18 100644
--- a/Octokit/Models/Response/GistComment.cs
+++ b/Octokit/Models/Response/GistComment.cs
@@ -9,9 +9,10 @@ public class GistComment
     {
         public GistComment() { }
 
-        public GistComment(int id, string url, string body, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt)
+        public GistComment(int id, string nodeId, string url, string body, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Body = body;
             User = user;
@@ -24,6 +25,11 @@ public GistComment(int id, string url, string body, User user, DateTimeOffset cr
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this gist comment.
         /// </summary>
diff --git a/Octokit/Models/Response/GistFork.cs b/Octokit/Models/Response/GistFork.cs
index 9729fc4307..888b81cb8c 100644
--- a/Octokit/Models/Response/GistFork.cs
+++ b/Octokit/Models/Response/GistFork.cs
@@ -9,13 +9,19 @@ public class GistFork
     {
         public GistFork() { }
 
-        public GistFork(User user, string url, DateTimeOffset createdAt)
+        public GistFork(string nodeId, User user, string url, DateTimeOffset createdAt)
         {
+            NodeId = nodeId;
             User = user;
             Url = url;
             CreatedAt = createdAt;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The <see cref="User"/> that created this <see cref="GistFork"/>
         /// </summary>

From 79011f882658e9d80d5ffa4d3d2eff81387dfe81 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 10:10:35 -0700
Subject: [PATCH 03/22] add node_id to Git Blob

---
 Octokit/Models/Response/Blob.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Blob.cs b/Octokit/Models/Response/Blob.cs
index 61af602889..b4df6a4f36 100644
--- a/Octokit/Models/Response/Blob.cs
+++ b/Octokit/Models/Response/Blob.cs
@@ -9,14 +9,20 @@ public class Blob
     {
         public Blob() { }
 
-        public Blob(string content, EncodingType encoding, string sha, int size)
+        public Blob(string nodeId, string content, EncodingType encoding, string sha, int size)
         {
+            NodeId = nodeId;
             Content = content;
             Encoding = encoding;
             Sha = sha;
             Size = size;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The content of the blob.
         /// </summary>

From c70ba807e1434131bcf668a4a48303e186fea16e Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 13:51:35 -0700
Subject: [PATCH 04/22] add node_id to Git Commit response

---
 Octokit/Models/Response/Commit.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Commit.cs b/Octokit/Models/Response/Commit.cs
index 1623692a48..c4764b3572 100644
--- a/Octokit/Models/Response/Commit.cs
+++ b/Octokit/Models/Response/Commit.cs
@@ -10,11 +10,12 @@ public class Commit : GitReference
     {
         public Commit() { }
 
-        public Commit(string url, string label, string @ref, string sha, User user, Repository repository, string message, Committer author, Committer committer, GitReference tree, IEnumerable<GitReference> parents, int commentCount, Verification verification)
+        public Commit(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, string message, Committer author, Committer committer, GitReference tree, IEnumerable<GitReference> parents, int commentCount, Verification verification)
             : base(url, label, @ref, sha, user, repository)
         {
             Ensure.ArgumentNotNull(parents, nameof(parents));
 
+            NodeId = nodeId;
             Message = message;
             Author = author;
             Committer = committer;
@@ -24,6 +25,11 @@ public Commit(string url, string label, string @ref, string sha, User user, Repo
             Verification = verification;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string Message { get; protected set; }
 
         public Committer Author { get; protected set; }

From da564f07f28e3d8ba3e66e4c40a371e27262be1e Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 13:52:58 -0700
Subject: [PATCH 05/22] add node_id to GitReference response

---
 Octokit/Models/Response/GitReference.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/GitReference.cs b/Octokit/Models/Response/GitReference.cs
index ec4466839f..8d9906917a 100644
--- a/Octokit/Models/Response/GitReference.cs
+++ b/Octokit/Models/Response/GitReference.cs
@@ -9,8 +9,9 @@ public class GitReference
     {
         public GitReference() { }
 
-        public GitReference(string url, string label, string @ref, string sha, User user, Repository repository)
+        public GitReference(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository)
         {
+            NodeId = nodeId;
             Url = url;
             Label = label;
             Ref = @ref;
@@ -19,6 +20,11 @@ public GitReference(string url, string label, string @ref, string sha, User user
             Repository = repository;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL associated with this reference.
         /// </summary>

From 05bf5ffb3cdbf24919c9e25188fb9eb286c9f64e Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Tue, 5 Jun 2018 13:57:28 -0700
Subject: [PATCH 06/22] add node_id to everything that inherits GitReference

---
 Octokit/Models/Response/Commit.cs       | 8 +-------
 Octokit/Models/Response/GitHubCommit.cs | 4 ++--
 Octokit/Models/Response/GitTag.cs       | 5 +++--
 Octokit/Models/Response/Merge.cs        | 4 ++--
 Octokit/Models/Response/TagObject.cs    | 4 ++--
 5 files changed, 10 insertions(+), 15 deletions(-)

diff --git a/Octokit/Models/Response/Commit.cs b/Octokit/Models/Response/Commit.cs
index c4764b3572..e72abbf649 100644
--- a/Octokit/Models/Response/Commit.cs
+++ b/Octokit/Models/Response/Commit.cs
@@ -11,11 +11,10 @@ public class Commit : GitReference
         public Commit() { }
 
         public Commit(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, string message, Committer author, Committer committer, GitReference tree, IEnumerable<GitReference> parents, int commentCount, Verification verification)
-            : base(url, label, @ref, sha, user, repository)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Ensure.ArgumentNotNull(parents, nameof(parents));
 
-            NodeId = nodeId;
             Message = message;
             Author = author;
             Committer = committer;
@@ -25,11 +24,6 @@ public Commit(string nodeId, string url, string label, string @ref, string sha,
             Verification = verification;
         }
 
-        /// <summary>
-        /// GraphQL Node Id
-        /// </summary>
-        public string NodeId { get; protected set; }
-
         public string Message { get; protected set; }
 
         public Committer Author { get; protected set; }
diff --git a/Octokit/Models/Response/GitHubCommit.cs b/Octokit/Models/Response/GitHubCommit.cs
index facc929b71..34f4a35606 100644
--- a/Octokit/Models/Response/GitHubCommit.cs
+++ b/Octokit/Models/Response/GitHubCommit.cs
@@ -11,8 +11,8 @@ public class GitHubCommit : GitReference
     {
         public GitHubCommit() { }
 
-        public GitHubCommit(string url, string label, string @ref, string sha, User user, Repository repository, Author author, string commentsUrl, Commit commit, Author committer, string htmlUrl, GitHubCommitStats stats, IReadOnlyList<GitReference> parents, IReadOnlyList<GitHubCommitFile> files)
-            : base(url, label, @ref, sha, user, repository)
+        public GitHubCommit(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, Author author, string commentsUrl, Commit commit, Author committer, string htmlUrl, GitHubCommitStats stats, IReadOnlyList<GitReference> parents, IReadOnlyList<GitHubCommitFile> files)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Author = author;
             CommentsUrl = commentsUrl;
diff --git a/Octokit/Models/Response/GitTag.cs b/Octokit/Models/Response/GitTag.cs
index 5c7e73d5d7..6db53711d7 100644
--- a/Octokit/Models/Response/GitTag.cs
+++ b/Octokit/Models/Response/GitTag.cs
@@ -7,8 +7,8 @@ public class GitTag : GitReference
     {
         public GitTag() { }
 
-        public GitTag(string url, string label, string @ref, string sha, User user, Repository repository, string tag, string message, Committer tagger, TagObject @object, Verification verification)
-            : base(url, label, @ref, sha, user, repository)
+        public GitTag(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, string tag, string message, Committer tagger, TagObject @object, Verification verification)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Tag = tag;
             Message = message;
@@ -17,6 +17,7 @@ public GitTag(string url, string label, string @ref, string sha, User user, Repo
             Verification = verification;
         }
 
+
         public string Tag { get; protected set; }
 
         public string Message { get; protected set; }
diff --git a/Octokit/Models/Response/Merge.cs b/Octokit/Models/Response/Merge.cs
index bfea87ef22..8af031a08b 100644
--- a/Octokit/Models/Response/Merge.cs
+++ b/Octokit/Models/Response/Merge.cs
@@ -10,8 +10,8 @@ public class Merge : GitReference
     {
         public Merge() { }
 
-        public Merge(string url, string label, string @ref, string sha, User user, Repository repository, Author author, Author committer, Commit commit, IEnumerable<GitReference> parents, string commentsUrl, int commentCount, string htmlUrl)
-            : base(url, label, @ref, sha, user, repository)
+        public Merge(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, Author author, Author committer, Commit commit, IEnumerable<GitReference> parents, string commentsUrl, int commentCount, string htmlUrl)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Ensure.ArgumentNotNull(parents, nameof(parents));
 
diff --git a/Octokit/Models/Response/TagObject.cs b/Octokit/Models/Response/TagObject.cs
index 26080b3b3d..0ffa976dfe 100644
--- a/Octokit/Models/Response/TagObject.cs
+++ b/Octokit/Models/Response/TagObject.cs
@@ -9,8 +9,8 @@ public class TagObject : GitReference
     {
         public TagObject() { }
 
-        public TagObject(string url, string label, string @ref, string sha, User user, Repository repository, TaggedType type)
-            : base(url, label, @ref, sha, user, repository)
+        public TagObject(string nodeId, string url, string label, string @ref, string sha, User user, Repository repository, TaggedType type)
+            : base(nodeId, url, label, @ref, sha, user, repository)
         {
             Type = type;
         }

From a1b7a3144a412a148b36d0d4edcebe4b116284cb Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Fri, 8 Jun 2018 10:24:04 -0700
Subject: [PATCH 07/22] add node_id to Issue

---
 Octokit/Models/Response/Issue.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Issue.cs b/Octokit/Models/Response/Issue.cs
index 2f7b930234..4a0add46fa 100644
--- a/Octokit/Models/Response/Issue.cs
+++ b/Octokit/Models/Response/Issue.cs
@@ -11,9 +11,10 @@ public class Issue
     {
         public Issue() { }
 
-        public Issue(string url, string htmlUrl, string commentsUrl, string eventsUrl, int number, ItemState state, string title, string body, User closedBy, User user, IReadOnlyList<Label> labels, User assignee, IReadOnlyList<User> assignees, Milestone milestone, int comments, PullRequest pullRequest, DateTimeOffset? closedAt, DateTimeOffset createdAt, DateTimeOffset? updatedAt, int id, bool locked, Repository repository, ReactionSummary reactions)
+        public Issue(string url, string htmlUrl, string commentsUrl, string eventsUrl, int number, ItemState state, string title, string body, User closedBy, User user, IReadOnlyList<Label> labels, User assignee, IReadOnlyList<User> assignees, Milestone milestone, int comments, PullRequest pullRequest, DateTimeOffset? closedAt, DateTimeOffset createdAt, DateTimeOffset? updatedAt, int id, string nodeId, bool locked, Repository repository, ReactionSummary reactions)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             HtmlUrl = htmlUrl;
             CommentsUrl = commentsUrl;
@@ -43,6 +44,11 @@ public Issue(string url, string htmlUrl, string commentsUrl, string eventsUrl, i
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this issue.
         /// </summary>

From 73317c387d253dfc712c1608dcb811eca57147f0 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:23:55 +1000
Subject: [PATCH 08/22] add node_id to IssueComment/IssueEvent

---
 Octokit/Models/Response/IssueComment.cs | 8 +++++++-
 Octokit/Models/Response/IssueEvent.cs   | 8 +++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/Octokit/Models/Response/IssueComment.cs b/Octokit/Models/Response/IssueComment.cs
index 0ed2cf76e0..989fe65ea7 100644
--- a/Octokit/Models/Response/IssueComment.cs
+++ b/Octokit/Models/Response/IssueComment.cs
@@ -10,9 +10,10 @@ public class IssueComment
     {
         public IssueComment() { }
 
-        public IssueComment(int id, string url, string htmlUrl, string body, DateTimeOffset createdAt, DateTimeOffset? updatedAt, User user, ReactionSummary reactions)
+        public IssueComment(int id, string nodeId, string url, string htmlUrl, string body, DateTimeOffset createdAt, DateTimeOffset? updatedAt, User user, ReactionSummary reactions)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             HtmlUrl = htmlUrl;
             Body = body;
@@ -27,6 +28,11 @@ public IssueComment(int id, string url, string htmlUrl, string body, DateTimeOff
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this issue comment.
         /// </summary>
diff --git a/Octokit/Models/Response/IssueEvent.cs b/Octokit/Models/Response/IssueEvent.cs
index 780700f11d..e26d50c080 100644
--- a/Octokit/Models/Response/IssueEvent.cs
+++ b/Octokit/Models/Response/IssueEvent.cs
@@ -9,9 +9,10 @@ public class IssueEvent
     {
         public IssueEvent() { }
 
-        public IssueEvent(int id, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt, Issue issue, string commitUrl)
+        public IssueEvent(int id, string nodeId, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt, Issue issue, string commitUrl)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Actor = actor;
             Assignee = assignee;
@@ -28,6 +29,11 @@ public IssueEvent(int id, string url, User actor, User assignee, Label label, Ev
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this issue/pull request event.
         /// </summary>

From d799ffbe5f1d1761c2c7925e593060a6c7c2e7a6 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:24:10 +1000
Subject: [PATCH 09/22] add node_id to Label

---
 Octokit/Models/Response/Label.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Label.cs b/Octokit/Models/Response/Label.cs
index e387963c34..095bd2a23b 100644
--- a/Octokit/Models/Response/Label.cs
+++ b/Octokit/Models/Response/Label.cs
@@ -8,10 +8,11 @@ public class Label
     {
         public Label() { }
 
-        public Label(string url, string name, string color, string description, bool @default)
+        public Label(string url, string name, string nodeId, string color, string description, bool @default)
         {
             Url = url;
             Name = name;
+            NodeId = nodeId;
             Color = color;
             Description = description;
             Default = @default;
@@ -27,6 +28,11 @@ public Label(string url, string name, string color, string description, bool @de
         /// </summary>
         public string Name { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Color of the label
         /// </summary>

From 9827a479a17cb39d63e74bd839d24f8340665814 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:24:21 +1000
Subject: [PATCH 10/22] add node_id to Milestone

---
 Octokit/Models/Response/Milestone.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Milestone.cs b/Octokit/Models/Response/Milestone.cs
index 2d82cf5328..3d1b77b755 100644
--- a/Octokit/Models/Response/Milestone.cs
+++ b/Octokit/Models/Response/Milestone.cs
@@ -14,11 +14,12 @@ public Milestone(int number)
             Number = number;
         }
 
-        public Milestone(string url, string htmlUrl, int number, ItemState state, string title, string description, User creator, int openIssues, int closedIssues, DateTimeOffset createdAt, DateTimeOffset? dueOn, DateTimeOffset? closedAt, DateTimeOffset? updatedAt)
+        public Milestone(string url, string htmlUrl, int number, string nodeId, ItemState state, string title, string description, User creator, int openIssues, int closedIssues, DateTimeOffset createdAt, DateTimeOffset? dueOn, DateTimeOffset? closedAt, DateTimeOffset? updatedAt)
         {
             Url = url;
             HtmlUrl = htmlUrl;
             Number = number;
+            NodeId = nodeId;
             State = state;
             Title = title;
             Description = description;
@@ -46,6 +47,11 @@ public Milestone(string url, string htmlUrl, int number, ItemState state, string
         /// </summary>
         public int Number { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Whether the milestone is open or closed.
         /// </summary>

From 1db226d14332b68af4969e43ae6e5db7717d937c Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:34:51 +1000
Subject: [PATCH 11/22] add node_id to Project/ProjectCard/ProjectColumn

---
 Octokit/Models/Response/Project.cs       | 8 +++++++-
 Octokit/Models/Response/ProjectCard.cs   | 8 +++++++-
 Octokit/Models/Response/ProjectColumn.cs | 8 +++++++-
 3 files changed, 21 insertions(+), 3 deletions(-)

diff --git a/Octokit/Models/Response/Project.cs b/Octokit/Models/Response/Project.cs
index c548e1dfa7..5552386b55 100644
--- a/Octokit/Models/Response/Project.cs
+++ b/Octokit/Models/Response/Project.cs
@@ -9,11 +9,12 @@ public class Project
     {
         public Project() { }
 
-        public Project(string ownerUrl, string url, int id, string name, string body, int number, ItemState state, User creator, DateTimeOffset createdAt, DateTimeOffset updatedAt)
+        public Project(string ownerUrl, string url, int id, string nodeId, string name, string body, int number, ItemState state, User creator, DateTimeOffset createdAt, DateTimeOffset updatedAt)
         {
             OwnerUrl = ownerUrl;
             Url = url;
             Id = id;
+            NodeId = nodeId;
             Name = name;
             Body = body;
             Number = number;
@@ -38,6 +39,11 @@ public Project(string ownerUrl, string url, int id, string name, string body, in
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The name for this project.
         /// </summary>
diff --git a/Octokit/Models/Response/ProjectCard.cs b/Octokit/Models/Response/ProjectCard.cs
index 8c313ec77f..ff8f139832 100644
--- a/Octokit/Models/Response/ProjectCard.cs
+++ b/Octokit/Models/Response/ProjectCard.cs
@@ -9,11 +9,12 @@ public class ProjectCard
     {
         public ProjectCard() { }
 
-        public ProjectCard(string columnUrl, string contentUrl, int id, string note, User creator, DateTimeOffset createdAt, DateTimeOffset updatedAt)
+        public ProjectCard(string columnUrl, string contentUrl, int id, string nodeId, string note, User creator, DateTimeOffset createdAt, DateTimeOffset updatedAt)
         {
             ColumnUrl = columnUrl;
             ContentUrl = contentUrl;
             Id = id;
+            NodeId = nodeId;
             Note = note;
             Creator = creator;
             CreatedAt = createdAt;
@@ -35,6 +36,11 @@ public ProjectCard(string columnUrl, string contentUrl, int id, string note, Use
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The note for this card.
         /// </summary>
diff --git a/Octokit/Models/Response/ProjectColumn.cs b/Octokit/Models/Response/ProjectColumn.cs
index 3b1de24981..98da4696c1 100644
--- a/Octokit/Models/Response/ProjectColumn.cs
+++ b/Octokit/Models/Response/ProjectColumn.cs
@@ -9,9 +9,10 @@ public class ProjectColumn
     {
         public ProjectColumn() { }
 
-        public ProjectColumn(int id, string name, string projectUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt)
+        public ProjectColumn(int id, string nodeId, string name, string projectUrl, DateTimeOffset createdAt, DateTimeOffset updatedAt)
         {
             Id = id;
+            NodeId = nodeId;
             Name = name;
             ProjectUrl = projectUrl;
             CreatedAt = createdAt;
@@ -23,6 +24,11 @@ public ProjectColumn(int id, string name, string projectUrl, DateTimeOffset crea
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The name for this column.
         /// </summary>

From 29ebaf50fa963095dce0e6c061d940128c460de4 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:37:03 +1000
Subject: [PATCH 12/22] add node_id to Reaction

---
 Octokit/Models/Response/Reaction.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Reaction.cs b/Octokit/Models/Response/Reaction.cs
index 0c10aa779a..e842d91618 100644
--- a/Octokit/Models/Response/Reaction.cs
+++ b/Octokit/Models/Response/Reaction.cs
@@ -30,9 +30,10 @@ public class Reaction
     {
         public Reaction() { }
 
-        public Reaction(int id, User user, ReactionType content)
+        public Reaction(int id, string nodeId, User user, ReactionType content)
         {
             Id = id;
+            NodeId = nodeId;
             User = user;
             Content = content;
         }
@@ -42,6 +43,11 @@ public Reaction(int id, User user, ReactionType content)
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Information about the user.
         /// </summary>

From 3787dbfa60b284b8fe4b954965d794da70762d00 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:42:38 +1000
Subject: [PATCH 13/22] add node_id to Release/ReleaseAsset

---
 Octokit/Models/Response/Release.cs      | 8 +++++++-
 Octokit/Models/Response/ReleaseAsset.cs | 8 +++++++-
 2 files changed, 14 insertions(+), 2 deletions(-)

diff --git a/Octokit/Models/Response/Release.cs b/Octokit/Models/Response/Release.cs
index 2fdca0d774..008a752e6c 100644
--- a/Octokit/Models/Response/Release.cs
+++ b/Octokit/Models/Response/Release.cs
@@ -11,13 +11,14 @@ public class Release
     {
         public Release() { }
 
-        public Release(string url, string htmlUrl, string assetsUrl, string uploadUrl, int id, string tagName, string targetCommitish, string name, string body, bool draft, bool prerelease, DateTimeOffset createdAt, DateTimeOffset? publishedAt, Author author, string tarballUrl, string zipballUrl, IReadOnlyList<ReleaseAsset> assets)
+        public Release(string url, string htmlUrl, string assetsUrl, string uploadUrl, int id, string nodeId, string tagName, string targetCommitish, string name, string body, bool draft, bool prerelease, DateTimeOffset createdAt, DateTimeOffset? publishedAt, Author author, string tarballUrl, string zipballUrl, IReadOnlyList<ReleaseAsset> assets)
         {
             Url = url;
             HtmlUrl = htmlUrl;
             AssetsUrl = assetsUrl;
             UploadUrl = uploadUrl;
             Id = id;
+            NodeId = nodeId;
             TagName = tagName;
             TargetCommitish = targetCommitish;
             Name = name;
@@ -48,6 +49,11 @@ public Release(string uploadUrl)
 
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string TagName { get; protected set; }
 
         public string TargetCommitish { get; protected set; }
diff --git a/Octokit/Models/Response/ReleaseAsset.cs b/Octokit/Models/Response/ReleaseAsset.cs
index c162fb174c..13b33a3955 100644
--- a/Octokit/Models/Response/ReleaseAsset.cs
+++ b/Octokit/Models/Response/ReleaseAsset.cs
@@ -9,10 +9,11 @@ public class ReleaseAsset
     {
         public ReleaseAsset() { }
 
-        public ReleaseAsset(string url, int id, string name, string label, string state, string contentType, int size, int downloadCount, DateTimeOffset createdAt, DateTimeOffset updatedAt, string browserDownloadUrl, Author uploader)
+        public ReleaseAsset(string url, int id, string nodeId, string name, string label, string state, string contentType, int size, int downloadCount, DateTimeOffset createdAt, DateTimeOffset updatedAt, string browserDownloadUrl, Author uploader)
         {
             Url = url;
             Id = id;
+            NodeId = nodeId;
             Name = name;
             Label = label;
             State = state;
@@ -29,6 +30,11 @@ public ReleaseAsset(string url, int id, string name, string label, string state,
 
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string Name { get; protected set; }
 
         public string Label { get; protected set; }

From 870c341f91fa8011f23449b8ce016e24f2bff561 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@tattsgroup.com>
Date: Sun, 10 Jun 2018 09:42:54 +1000
Subject: [PATCH 14/22] add node_id to Team

---
 Octokit/Models/Response/Team.cs | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/Octokit/Models/Response/Team.cs b/Octokit/Models/Response/Team.cs
index 3b5a8fe374..b136df3217 100644
--- a/Octokit/Models/Response/Team.cs
+++ b/Octokit/Models/Response/Team.cs
@@ -12,10 +12,11 @@ public class Team
     {
         public Team() { }
 
-        public Team(string url, int id, string name, string description, TeamPrivacy privacy, Permission permission, int membersCount, int reposCount, Organization organization, Team parent, string ldapDistinguishedName)
+        public Team(string url, int id, string nodeId, string name, string description, TeamPrivacy privacy, Permission permission, int membersCount, int reposCount, Organization organization, Team parent, string ldapDistinguishedName)
         {
             Url = url;
             Id = id;
+            NodeId = nodeId;
             Name = name;
             Description = description;
             Privacy = privacy;
@@ -37,6 +38,11 @@ public Team(string url, int id, string name, string description, TeamPrivacy pri
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// team name
         /// </summary>

From 59c770b533ba2d89baaaf88489e2276843c60ef4 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 11 Jun 2018 16:43:18 +1000
Subject: [PATCH 15/22] add node_id to
 Repository.RepositoryContributor/RepositoryInvitation/RepositoryTag

---
 Octokit/Models/Response/Repository.cs            | 8 +++++++-
 Octokit/Models/Response/RepositoryContributor.cs | 4 ++--
 Octokit/Models/Response/RepositoryInvitation.cs  | 8 +++++++-
 Octokit/Models/Response/RepositoryTag.cs         | 8 +++++++-
 4 files changed, 23 insertions(+), 5 deletions(-)

diff --git a/Octokit/Models/Response/Repository.cs b/Octokit/Models/Response/Repository.cs
index 77d87140c1..65a2716f7f 100644
--- a/Octokit/Models/Response/Repository.cs
+++ b/Octokit/Models/Response/Repository.cs
@@ -14,7 +14,7 @@ public Repository(long id)
             Id = id;
         }
 
-        public Repository(string url, string htmlUrl, string cloneUrl, string gitUrl, string sshUrl, string svnUrl, string mirrorUrl, long id, User owner, string name, string fullName, string description, string homepage, string language, bool @private, bool fork, int forksCount, int stargazersCount, string defaultBranch, int openIssuesCount, DateTimeOffset? pushedAt, DateTimeOffset createdAt, DateTimeOffset updatedAt, RepositoryPermissions permissions, Repository parent, Repository source, LicenseMetadata license, bool hasIssues, bool hasWiki, bool hasDownloads, bool hasPages, int subscribersCount, long size, bool? allowRebaseMerge, bool? allowSquashMerge, bool? allowMergeCommit)
+        public Repository(string url, string htmlUrl, string cloneUrl, string gitUrl, string sshUrl, string svnUrl, string mirrorUrl, long id, string nodeId, User owner, string name, string fullName, string description, string homepage, string language, bool @private, bool fork, int forksCount, int stargazersCount, string defaultBranch, int openIssuesCount, DateTimeOffset? pushedAt, DateTimeOffset createdAt, DateTimeOffset updatedAt, RepositoryPermissions permissions, Repository parent, Repository source, LicenseMetadata license, bool hasIssues, bool hasWiki, bool hasDownloads, bool hasPages, int subscribersCount, long size, bool? allowRebaseMerge, bool? allowSquashMerge, bool? allowMergeCommit)
         {
             Url = url;
             HtmlUrl = htmlUrl;
@@ -24,6 +24,7 @@ public Repository(string url, string htmlUrl, string cloneUrl, string gitUrl, st
             SvnUrl = svnUrl;
             MirrorUrl = mirrorUrl;
             Id = id;
+            NodeId = nodeId;
             Owner = owner;
             Name = name;
             FullName = fullName;
@@ -70,6 +71,11 @@ public Repository(string url, string htmlUrl, string cloneUrl, string gitUrl, st
 
         public long Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public User Owner { get; protected set; }
 
         public string Name { get; protected set; }
diff --git a/Octokit/Models/Response/RepositoryContributor.cs b/Octokit/Models/Response/RepositoryContributor.cs
index eabbbe7592..2c3d0e3262 100644
--- a/Octokit/Models/Response/RepositoryContributor.cs
+++ b/Octokit/Models/Response/RepositoryContributor.cs
@@ -10,8 +10,8 @@ public class RepositoryContributor : Author
     {
         public RepositoryContributor() { }
 
-        public RepositoryContributor(string login, int id, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin, int contributions)
-            : base(login, id, avatarUrl, url, htmlUrl, followersUrl, followingUrl, gistsUrl, type, starredUrl, subscriptionsUrl, organizationsUrl, reposUrl, eventsUrl, receivedEventsUrl, siteAdmin)
+        public RepositoryContributor(string login, int id, string nodeId, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin, int contributions)
+            : base(login, id, nodeId, avatarUrl, url, htmlUrl, followersUrl, followingUrl, gistsUrl, type, starredUrl, subscriptionsUrl, organizationsUrl, reposUrl, eventsUrl, receivedEventsUrl, siteAdmin)
         {
             Contributions = contributions;
         }
diff --git a/Octokit/Models/Response/RepositoryInvitation.cs b/Octokit/Models/Response/RepositoryInvitation.cs
index d28655a878..fe5a86fcda 100644
--- a/Octokit/Models/Response/RepositoryInvitation.cs
+++ b/Octokit/Models/Response/RepositoryInvitation.cs
@@ -22,9 +22,10 @@ public class RepositoryInvitation
     {
         public RepositoryInvitation() { }
 
-        public RepositoryInvitation(int id, Repository repository, User invitee, User inviter, InvitationPermissionType permissions, DateTimeOffset createdAt, string url, string htmlUrl)
+        public RepositoryInvitation(int id, string nodeId, Repository repository, User invitee, User inviter, InvitationPermissionType permissions, DateTimeOffset createdAt, string url, string htmlUrl)
         {
             Id = id;
+            NodeId = nodeId;
             Repository = repository;
             Invitee = invitee;
             Inviter = inviter;
@@ -36,6 +37,11 @@ public RepositoryInvitation(int id, Repository repository, User invitee, User in
 
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public Repository Repository { get; protected set; }
 
         public User Invitee { get; protected set; }
diff --git a/Octokit/Models/Response/RepositoryTag.cs b/Octokit/Models/Response/RepositoryTag.cs
index 835e12a91f..37ce9baf20 100644
--- a/Octokit/Models/Response/RepositoryTag.cs
+++ b/Octokit/Models/Response/RepositoryTag.cs
@@ -11,9 +11,10 @@ public RepositoryTag() { }
 
         [SuppressMessage("Microsoft.Naming", "CA1704:IdentifiersShouldBeSpelledCorrectly", MessageId = "tarball")]
         [SuppressMessage("Microsoft.Naming", "CA1704:IdentifiersShouldBeSpelledCorrectly", MessageId = "zipball")]
-        public RepositoryTag(string name, GitReference commit, string zipballUrl, string tarballUrl)
+        public RepositoryTag(string name, string nodeId, GitReference commit, string zipballUrl, string tarballUrl)
         {
             Name = name;
+            NodeId = nodeId;
             Commit = commit;
             ZipballUrl = zipballUrl;
             TarballUrl = tarballUrl;
@@ -21,6 +22,11 @@ public RepositoryTag(string name, GitReference commit, string zipballUrl, string
 
         public string Name { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public GitReference Commit { get; protected set; }
 
         [SuppressMessage("Microsoft.Naming", "CA1704:IdentifiersShouldBeSpelledCorrectly", MessageId = "Zipball")]

From 9171625316d1c1f340d036e273033a8f6181f963 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 11 Jun 2018 16:43:53 +1000
Subject: [PATCH 16/22] add node_id to Commit related responses

---
 Octokit/Models/Common/Committer.cs       | 9 ++++++++-
 Octokit/Models/Response/Author.cs        | 8 +++++++-
 Octokit/Models/Response/CommitComment.cs | 8 +++++++-
 Octokit/Models/Response/CommitStatus.cs  | 8 +++++++-
 4 files changed, 29 insertions(+), 4 deletions(-)

diff --git a/Octokit/Models/Common/Committer.cs b/Octokit/Models/Common/Committer.cs
index c9949d5487..e8a3d6871f 100644
--- a/Octokit/Models/Common/Committer.cs
+++ b/Octokit/Models/Common/Committer.cs
@@ -19,16 +19,23 @@ public Committer() { }
         /// <summary>
         /// Initializes a new instance of the <see cref="Committer"/> class.
         /// </summary>
+        /// <param name="nodeId">The GraphQL Node Id</param>
         /// <param name="name">The full name of the author or committer.</param>
         /// <param name="email">The email.</param>
         /// <param name="date">The date.</param>
-        public Committer(string name, string email, DateTimeOffset date)
+        public Committer(string nodeId, string name, string email, DateTimeOffset date)
         {
+            NodeId = nodeId;
             Name = name;
             Email = email;
             Date = date;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Gets the name of the author or committer.
         /// </summary>
diff --git a/Octokit/Models/Response/Author.cs b/Octokit/Models/Response/Author.cs
index a416b7bb44..c37ee6a6e8 100644
--- a/Octokit/Models/Response/Author.cs
+++ b/Octokit/Models/Response/Author.cs
@@ -9,10 +9,11 @@ public class Author
     {
         public Author() { }
 
-        public Author(string login, int id, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin)
+        public Author(string login, int id, string nodeId, string avatarUrl, string url, string htmlUrl, string followersUrl, string followingUrl, string gistsUrl, string type, string starredUrl, string subscriptionsUrl, string organizationsUrl, string reposUrl, string eventsUrl, string receivedEventsUrl, bool siteAdmin)
         {
             Login = login;
             Id = id;
+            NodeId = nodeId;
             AvatarUrl = avatarUrl;
             Url = url;
             HtmlUrl = htmlUrl;
@@ -33,6 +34,11 @@ public Author(string login, int id, string avatarUrl, string url, string htmlUrl
 
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string AvatarUrl { get; protected set; }
 
         public string Url { get; protected set; }
diff --git a/Octokit/Models/Response/CommitComment.cs b/Octokit/Models/Response/CommitComment.cs
index 4038f6e4e8..531423a103 100644
--- a/Octokit/Models/Response/CommitComment.cs
+++ b/Octokit/Models/Response/CommitComment.cs
@@ -9,9 +9,10 @@ public class CommitComment
     {
         public CommitComment() { }
 
-        public CommitComment(int id, string url, string htmlUrl, string body, string path, int position, int? line, string commitId, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt, ReactionSummary reactions)
+        public CommitComment(int id, string nodeId, string url, string htmlUrl, string body, string path, int position, int? line, string commitId, User user, DateTimeOffset createdAt, DateTimeOffset? updatedAt, ReactionSummary reactions)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             HtmlUrl = htmlUrl;
             Body = body;
@@ -30,6 +31,11 @@ public CommitComment(int id, string url, string htmlUrl, string body, string pat
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this repository comment.
         /// </summary>
diff --git a/Octokit/Models/Response/CommitStatus.cs b/Octokit/Models/Response/CommitStatus.cs
index dfc955c50d..d5660f21da 100644
--- a/Octokit/Models/Response/CommitStatus.cs
+++ b/Octokit/Models/Response/CommitStatus.cs
@@ -10,7 +10,7 @@ public class CommitStatus
     {
         public CommitStatus() { }
 
-        public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitState state, string targetUrl, string description, string context, long id, string url, User creator)
+        public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitState state, string targetUrl, string description, string context, long id, string nodeId, string url, User creator)
         {
             CreatedAt = createdAt;
             UpdatedAt = updatedAt;
@@ -19,6 +19,7 @@ public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitSt
             Description = description;
             Context = context;
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Creator = creator;
         }
@@ -59,6 +60,11 @@ public CommitStatus(DateTimeOffset createdAt, DateTimeOffset updatedAt, CommitSt
         /// </summary>
         public long Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL of the status.
         /// </summary>

From 3a898711482289b3327fade91b1c09d69e6dbbd7 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 11 Jun 2018 16:44:14 +1000
Subject: [PATCH 17/22] Add node_id to PullRequest related responses

---
 Octokit/Models/Response/PullRequest.cs              | 8 +++++++-
 Octokit/Models/Response/PullRequestCommit.cs        | 8 +++++++-
 Octokit/Models/Response/PullRequestReview.cs        | 8 +++++++-
 Octokit/Models/Response/PullRequestReviewComment.cs | 8 +++++++-
 4 files changed, 28 insertions(+), 4 deletions(-)

diff --git a/Octokit/Models/Response/PullRequest.cs b/Octokit/Models/Response/PullRequest.cs
index 115f420654..99fab52019 100644
--- a/Octokit/Models/Response/PullRequest.cs
+++ b/Octokit/Models/Response/PullRequest.cs
@@ -16,9 +16,10 @@ public PullRequest(int number)
             Number = number;
         }
 
-        public PullRequest(long id, string url, string htmlUrl, string diffUrl, string patchUrl, string issueUrl, string statusesUrl, int number, ItemState state, string title, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, DateTimeOffset? closedAt, DateTimeOffset? mergedAt, GitReference head, GitReference @base, User user, User assignee, IReadOnlyList<User> assignees, bool? mergeable, MergeableState? mergeableState, User mergedBy, string mergeCommitSha, int comments, int commits, int additions, int deletions, int changedFiles, Milestone milestone, bool locked, bool? maintainerCanModify, IReadOnlyList<User> requestedReviewers)
+        public PullRequest(long id, string nodeId, string url, string htmlUrl, string diffUrl, string patchUrl, string issueUrl, string statusesUrl, int number, ItemState state, string title, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, DateTimeOffset? closedAt, DateTimeOffset? mergedAt, GitReference head, GitReference @base, User user, User assignee, IReadOnlyList<User> assignees, bool? mergeable, MergeableState? mergeableState, User mergedBy, string mergeCommitSha, int comments, int commits, int additions, int deletions, int changedFiles, Milestone milestone, bool locked, bool? maintainerCanModify, IReadOnlyList<User> requestedReviewers)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             HtmlUrl = htmlUrl;
             DiffUrl = diffUrl;
@@ -58,6 +59,11 @@ public PullRequest(long id, string url, string htmlUrl, string diffUrl, string p
         /// </summary>
         public long Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this pull request.
         /// </summary>
diff --git a/Octokit/Models/Response/PullRequestCommit.cs b/Octokit/Models/Response/PullRequestCommit.cs
index fca85bfaaa..d25c561cb4 100644
--- a/Octokit/Models/Response/PullRequestCommit.cs
+++ b/Octokit/Models/Response/PullRequestCommit.cs
@@ -11,10 +11,11 @@ public class PullRequestCommit
     {
         public PullRequestCommit() { }
 
-        public PullRequestCommit(Committer author, string commentsUrl, Commit commit, Committer committer, string htmlUrl, IEnumerable<GitReference> parents, string sha, string url)
+        public PullRequestCommit(string nodeId, Committer author, string commentsUrl, Commit commit, Committer committer, string htmlUrl, IEnumerable<GitReference> parents, string sha, string url)
         {
             Ensure.ArgumentNotNull(parents, nameof(parents));
 
+            NodeId = nodeId;
             Author = author;
             CommentsUrl = commentsUrl;
             Commit = commit;
@@ -25,6 +26,11 @@ public PullRequestCommit(Committer author, string commentsUrl, Commit commit, Co
             Url = url;
         }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public Committer Author { get; protected set; }
 
         public string CommentsUrl { get; protected set; }
diff --git a/Octokit/Models/Response/PullRequestReview.cs b/Octokit/Models/Response/PullRequestReview.cs
index 49180be48f..e6e1d33c40 100644
--- a/Octokit/Models/Response/PullRequestReview.cs
+++ b/Octokit/Models/Response/PullRequestReview.cs
@@ -15,9 +15,10 @@ public PullRequestReview(long id)
             Id = id;
         }
 
-        public PullRequestReview(long id, string commitId, User user, string body, string htmlUrl, string pullRequestUrl, PullRequestReviewState state)
+        public PullRequestReview(long id, string nodeId, string commitId, User user, string body, string htmlUrl, string pullRequestUrl, PullRequestReviewState state)
         {
             Id = id;
+            NodeId = nodeId;
             CommitId = commitId;
             User = user;
             Body = body;
@@ -31,6 +32,11 @@ public PullRequestReview(long id, string commitId, User user, string body, strin
         /// </summary>
         public long Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The state of the review
         /// </summary>
diff --git a/Octokit/Models/Response/PullRequestReviewComment.cs b/Octokit/Models/Response/PullRequestReviewComment.cs
index af78cf313f..d6a71d424a 100644
--- a/Octokit/Models/Response/PullRequestReviewComment.cs
+++ b/Octokit/Models/Response/PullRequestReviewComment.cs
@@ -15,11 +15,12 @@ public PullRequestReviewComment(int id)
             Id = id;
         }
 
-        public PullRequestReviewComment(string url, int id, string diffHunk, string path, int? position, int? originalPosition, string commitId, string originalCommitId, User user, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, string htmlUrl, string pullRequestUrl, ReactionSummary reactions, int? inReplyToId, int? pullRequestReviewId)
+        public PullRequestReviewComment(string url, int id, string nodeId, string diffHunk, string path, int? position, int? originalPosition, string commitId, string originalCommitId, User user, string body, DateTimeOffset createdAt, DateTimeOffset updatedAt, string htmlUrl, string pullRequestUrl, ReactionSummary reactions, int? inReplyToId, int? pullRequestReviewId)
         {
             PullRequestReviewId = pullRequestReviewId;
             Url = url;
             Id = id;
+            NodeId = nodeId;
             DiffHunk = diffHunk;
             Path = path;
             Position = position;
@@ -46,6 +47,11 @@ public PullRequestReviewComment(string url, int id, string diffHunk, string path
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The diff hunk the comment is about.
         /// </summary>

From 9ceb3982079825fb01d7812f451543723b6673e1 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 11 Jun 2018 16:45:22 +1000
Subject: [PATCH 18/22] Add node_id to any response models it was found to be
 missing, based on auditing integration test responses

---
 Octokit/Models/Response/EventInfo.cs                |  8 +++++++-
 Octokit/Models/Response/License.cs                  | 12 ++----------
 Octokit/Models/Response/LicenseMetadata.cs          | 13 +++++++------
 Octokit/Models/Response/Migration.cs                |  7 +++++++
 .../Response/OrganizationMembershipInvitation.cs    |  9 ++++++++-
 Octokit/Models/Response/Reference.cs                |  8 +++++++-
 Octokit/Models/Response/TimelineEventInfo.cs        |  9 ++++++++-
 7 files changed, 46 insertions(+), 20 deletions(-)

diff --git a/Octokit/Models/Response/EventInfo.cs b/Octokit/Models/Response/EventInfo.cs
index 5491acb8bd..7daf2e15bb 100644
--- a/Octokit/Models/Response/EventInfo.cs
+++ b/Octokit/Models/Response/EventInfo.cs
@@ -11,9 +11,10 @@ public class EventInfo
     {
         public EventInfo() { }
 
-        public EventInfo(int id, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt)
+        public EventInfo(int id, string nodeId, string url, User actor, User assignee, Label label, EventInfoState @event, string commitId, DateTimeOffset createdAt)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Actor = actor;
             Assignee = assignee;
@@ -28,6 +29,11 @@ public EventInfo(int id, string url, User actor, User assignee, Label label, Eve
         /// </summary>
         public int Id { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// The URL for this event.
         /// </summary>
diff --git a/Octokit/Models/Response/License.cs b/Octokit/Models/Response/License.cs
index c5e9d7188a..0bd31205bd 100644
--- a/Octokit/Models/Response/License.cs
+++ b/Octokit/Models/Response/License.cs
@@ -11,6 +11,7 @@ public class License : LicenseMetadata
     {
         public License(
             string key,
+            string nodeId,
             string name,
             string spdxId,
             string url,
@@ -22,17 +23,8 @@ public License(
             string body,
             IEnumerable<string> required,
             IEnumerable<string> permitted,
-            IEnumerable<string> forbidden) : base(key, name, spdxId, url, featured)
+            IEnumerable<string> forbidden) : base(key, nodeId, name, spdxId, url, featured)
         {
-            Ensure.ArgumentNotNull(htmlUrl, nameof(htmlUrl));
-            Ensure.ArgumentNotNull(description, nameof(description));
-            Ensure.ArgumentNotNull(category, nameof(category));
-            Ensure.ArgumentNotNull(implementation, nameof(implementation));
-            Ensure.ArgumentNotNull(body, nameof(body));
-            Ensure.ArgumentNotNull(required, nameof(required));
-            Ensure.ArgumentNotNull(permitted, nameof(permitted));
-            Ensure.ArgumentNotNull(forbidden, nameof(forbidden));
-
             HtmlUrl = htmlUrl;
             Description = description;
             Category = category;
diff --git a/Octokit/Models/Response/LicenseMetadata.cs b/Octokit/Models/Response/LicenseMetadata.cs
index 2b066342f8..d00c075088 100644
--- a/Octokit/Models/Response/LicenseMetadata.cs
+++ b/Octokit/Models/Response/LicenseMetadata.cs
@@ -6,14 +6,10 @@ namespace Octokit
     [DebuggerDisplay("{DebuggerDisplay,nq}")]
     public class LicenseMetadata
     {
-        public LicenseMetadata(string key, string name, string spdxId, string url, bool featured)
+        public LicenseMetadata(string key, string nodeId, string name, string spdxId, string url, bool featured)
         {
-            Ensure.ArgumentNotNullOrEmptyString(key, nameof(key));
-            Ensure.ArgumentNotNullOrEmptyString(name, nameof(name));
-            Ensure.ArgumentNotNullOrEmptyString(spdxId, nameof(spdxId));
-            Ensure.ArgumentNotNull(url, nameof(url));
-
             Key = key;
+            NodeId = nodeId;
             Name = name;
             SpdxId = spdxId;
             Url = url;
@@ -29,6 +25,11 @@ public LicenseMetadata()
         /// </summary>
         public string Key { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Friendly name of the license.
         /// </summary>
diff --git a/Octokit/Models/Response/Migration.cs b/Octokit/Models/Response/Migration.cs
index 84c960baa4..f56213b1ac 100644
--- a/Octokit/Models/Response/Migration.cs
+++ b/Octokit/Models/Response/Migration.cs
@@ -23,6 +23,7 @@ public Migration()
 
         public Migration(
             int id,
+            string nodeId,
             string guid,
             MigrationState state,
             bool lockRepositories,
@@ -33,6 +34,7 @@ public Migration(
             IReadOnlyList<Repository> repositories)
         {
             Id = id;
+            NodeId = nodeId;
             Guid = guid;
             State = state;
             LockRepositories = lockRepositories;
@@ -48,6 +50,11 @@ public Migration(
         /// </summary>
         public int Id { get; private set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         /// <summary>
         /// Guid of migration.
         /// </summary>
diff --git a/Octokit/Models/Response/OrganizationMembershipInvitation.cs b/Octokit/Models/Response/OrganizationMembershipInvitation.cs
index 7d5db50056..5413cbe024 100644
--- a/Octokit/Models/Response/OrganizationMembershipInvitation.cs
+++ b/Octokit/Models/Response/OrganizationMembershipInvitation.cs
@@ -12,9 +12,10 @@ public OrganizationMembershipInvitation()
         {
         }
 
-        public OrganizationMembershipInvitation(int id, string login, string email, OrganizationMembershipRole role, DateTimeOffset createdAt, User inviter)
+        public OrganizationMembershipInvitation(int id, string nodeId, string login, string email, OrganizationMembershipRole role, DateTimeOffset createdAt, User inviter)
         {
             Id = id;
+            NodeId = nodeId;
             Login = login;
             Email = email;
             Role = role;
@@ -23,6 +24,12 @@ public OrganizationMembershipInvitation(int id, string login, string email, Orga
         }
 
         public int Id { get; protected set; }
+
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string Login { get; protected set; }
         public string Email { get; protected set; }
         public StringEnum<OrganizationMembershipRole> Role { get; protected set; }
diff --git a/Octokit/Models/Response/Reference.cs b/Octokit/Models/Response/Reference.cs
index ce52179868..9cde85b692 100644
--- a/Octokit/Models/Response/Reference.cs
+++ b/Octokit/Models/Response/Reference.cs
@@ -8,15 +8,21 @@ public class Reference
     {
         public Reference() { }
 
-        public Reference(string @ref, string url, TagObject @object)
+        public Reference(string @ref, string nodeId, string url, TagObject @object)
         {
             Ref = @ref;
+            NodeId = nodeId;
             Url = url;
             Object = @object;
         }
 
         public string Ref { get; protected set; }
 
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string Url { get; protected set; }
 
         public TagObject Object { get; protected set; }
diff --git a/Octokit/Models/Response/TimelineEventInfo.cs b/Octokit/Models/Response/TimelineEventInfo.cs
index fcf7c10793..081a320177 100644
--- a/Octokit/Models/Response/TimelineEventInfo.cs
+++ b/Octokit/Models/Response/TimelineEventInfo.cs
@@ -9,9 +9,10 @@ public class TimelineEventInfo
     {
         public TimelineEventInfo() { }
 
-        public TimelineEventInfo(int id, string url, User actor, string commitId, EventInfoState @event, DateTimeOffset createdAt, Label label, User assignee, Milestone milestone, SourceInfo source, RenameInfo rename)
+        public TimelineEventInfo(int id, string nodeId, string url, User actor, string commitId, EventInfoState @event, DateTimeOffset createdAt, Label label, User assignee, Milestone milestone, SourceInfo source, RenameInfo rename)
         {
             Id = id;
+            NodeId = nodeId;
             Url = url;
             Actor = actor;
             CommitId = commitId;
@@ -25,6 +26,12 @@ public TimelineEventInfo(int id, string url, User actor, string commitId, EventI
         }
 
         public int Id { get; protected set; }
+
+        /// <summary>
+        /// GraphQL Node Id
+        /// </summary>
+        public string NodeId { get; protected set; }
+
         public string Url { get; protected set; }
         public User Actor { get; protected set; }
         public string CommitId { get; protected set; }

From d87ab0ea7d786e8aef60901236cd96288c947681 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 11 Jun 2018 16:46:24 +1000
Subject: [PATCH 19/22] remove unused test variable that was using a response
 ctor

---
 Octokit.Tests/Models/MigrationTests.cs | 14 --------------
 1 file changed, 14 deletions(-)

diff --git a/Octokit.Tests/Models/MigrationTests.cs b/Octokit.Tests/Models/MigrationTests.cs
index 31b89da75e..128606dc13 100644
--- a/Octokit.Tests/Models/MigrationTests.cs
+++ b/Octokit.Tests/Models/MigrationTests.cs
@@ -110,20 +110,6 @@ public class MigrationTests
               ]
             }";
 
-        private static readonly Migration migration = new Migration(
-            id: 79,
-            guid: "0b989ba4-242f-11e5-81e1-c7b6966d2516",
-            state: Migration.MigrationState.Exported,
-            lockRepositories: true,
-            excludeAttachments: false,
-            url: "https://api.github.com/orgs/octo-org/migrations/79",
-            createdAt: "2015-07-06T15:33:38-07:00",
-            updatedAt: "2015-07-06T15:33:38-07:00",
-            repositories: new List<Repository>
-            {
-                new Repository(1296269)
-            });
-
         [Fact]
         public void CanBeDeserialized()
         {

From d091bbbd7677508d8bc36e3ddf958496803951a4 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Wed, 13 Jun 2018 09:52:00 +1000
Subject: [PATCH 20/22] fix tests that need to handle node_id now

---
 Octokit.Tests/Clients/TagsClientTests.cs                    | 3 ++-
 Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs | 4 ++--
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/Octokit.Tests/Clients/TagsClientTests.cs b/Octokit.Tests/Clients/TagsClientTests.cs
index ec697e7abc..6e754ac521 100644
--- a/Octokit.Tests/Clients/TagsClientTests.cs
+++ b/Octokit.Tests/Clients/TagsClientTests.cs
@@ -113,7 +113,7 @@ public void PerformsNewTagSerialization()
                 Tag = "tag-name",
                 Object = "tag-object",
                 Type = TaggedType.Tree,
-                Tagger = new Committer("tagger-name", "tagger-email", DateTimeOffset.Parse("2013-09-03T13:42:52Z"))
+                Tagger = new Committer("123ABC", "tagger-name", "tagger-email", DateTimeOffset.Parse("2013-09-03T13:42:52Z"))
             };
 
             var json = new SimpleJsonSerializer().Serialize(tag);
@@ -123,6 +123,7 @@ public void PerformsNewTagSerialization()
                                             "\"object\":\"tag-object\"," +
                                             "\"type\":\"tree\"," +
                                             "\"tagger\":{" +
+                                                "\"node_id\":\"123ABC\"," +
                                                 "\"name\":\"tagger-name\"," +
                                                 "\"email\":\"tagger-email\"," +
                                                 "\"date\":\"2013-09-03T13:42:52Z\"" +
diff --git a/Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs b/Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs
index 8ca0e56061..b76e89bdf2 100644
--- a/Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs
+++ b/Octokit.Tests/Reactive/ObservablePullRequestsClientTests.cs
@@ -629,7 +629,7 @@ public class TheCommitsMethod
             [Fact]
             public async Task FetchesAllCommitsForPullRequest()
             {
-                var commit = new PullRequestCommit(null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
+                var commit = new PullRequestCommit("123ABC", null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
                 var expectedUrl = "repos/fake/repo/pulls/42/commits";
                 var gitHubClient = Substitute.For<IGitHubClient>();
                 var connection = Substitute.For<IConnection>();
@@ -653,7 +653,7 @@ public async Task FetchesAllCommitsForPullRequest()
             [Fact]
             public async Task FetchesAllCommitsForPullRequestWithRepositoryId()
             {
-                var commit = new PullRequestCommit(null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
+                var commit = new PullRequestCommit("123ABC", null, null, null, null, null, Enumerable.Empty<GitReference>(), null, null);
                 var expectedUrl = "repositories/1/pulls/42/commits";
                 var gitHubClient = Substitute.For<IGitHubClient>();
                 var connection = Substitute.For<IConnection>();

From 67fc133b86584799eb610eee9344abf1e798c808 Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Mon, 25 Jun 2018 23:52:17 +1000
Subject: [PATCH 21/22] Committer is a request object as well as response, so
 make nodeId optional

---
 Octokit.Tests/Clients/TagsClientTests.cs |  2 +-
 Octokit/Models/Common/Committer.cs       | 13 +++++++++++++
 2 files changed, 14 insertions(+), 1 deletion(-)

diff --git a/Octokit.Tests/Clients/TagsClientTests.cs b/Octokit.Tests/Clients/TagsClientTests.cs
index 6e754ac521..fc18d520fd 100644
--- a/Octokit.Tests/Clients/TagsClientTests.cs
+++ b/Octokit.Tests/Clients/TagsClientTests.cs
@@ -113,7 +113,7 @@ public void PerformsNewTagSerialization()
                 Tag = "tag-name",
                 Object = "tag-object",
                 Type = TaggedType.Tree,
-                Tagger = new Committer("123ABC", "tagger-name", "tagger-email", DateTimeOffset.Parse("2013-09-03T13:42:52Z"))
+                Tagger = new Committer("tagger-name", "tagger-email", DateTimeOffset.Parse("2013-09-03T13:42:52Z"))
             };
 
             var json = new SimpleJsonSerializer().Serialize(tag);
diff --git a/Octokit/Models/Common/Committer.cs b/Octokit/Models/Common/Committer.cs
index e8a3d6871f..bc8539d0c3 100644
--- a/Octokit/Models/Common/Committer.cs
+++ b/Octokit/Models/Common/Committer.cs
@@ -16,6 +16,19 @@ public class Committer
         /// </summary>
         public Committer() { }
 
+        /// <summary>
+        /// Initializes a new instance of the <see cref="Committer"/> class.
+        /// </summary>
+        /// <param name="name">The full name of the author or committer.</param>
+        /// <param name="email">The email.</param>
+        /// <param name="date">The date.</param>
+        public Committer(string name, string email, DateTimeOffset date)
+        {
+            Name = name;
+            Email = email;
+            Date = date;
+        }
+
         /// <summary>
         /// Initializes a new instance of the <see cref="Committer"/> class.
         /// </summary>

From 423c201dceb9ce5ad5ddf7e846f9d0b3c3c8818a Mon Sep 17 00:00:00 2001
From: Ryan Gribble <ryan.gribble@gmail.com>
Date: Tue, 26 Jun 2018 13:51:09 +1000
Subject: [PATCH 22/22] fix test

---
 Octokit.Tests/Clients/TagsClientTests.cs | 1 -
 1 file changed, 1 deletion(-)

diff --git a/Octokit.Tests/Clients/TagsClientTests.cs b/Octokit.Tests/Clients/TagsClientTests.cs
index fc18d520fd..ec697e7abc 100644
--- a/Octokit.Tests/Clients/TagsClientTests.cs
+++ b/Octokit.Tests/Clients/TagsClientTests.cs
@@ -123,7 +123,6 @@ public void PerformsNewTagSerialization()
                                             "\"object\":\"tag-object\"," +
                                             "\"type\":\"tree\"," +
                                             "\"tagger\":{" +
-                                                "\"node_id\":\"123ABC\"," +
                                                 "\"name\":\"tagger-name\"," +
                                                 "\"email\":\"tagger-email\"," +
                                                 "\"date\":\"2013-09-03T13:42:52Z\"" +