Update octokit.net to 0.29.0

Author: grokys

src/GitHub.App/Api/ApiClient.cs

@@ -269,11 +269,7 @@ public IObservable<Branch> GetBranches(string owner, string repo)
             Guard.ArgumentNotEmptyString(owner, nameof(owner));
             Guard.ArgumentNotEmptyString(repo, nameof(repo));
 
-#pragma warning disable 618
-            // GetAllBranches is obsolete, but don't want to introduce the change to fix the
-            // warning in the PR, so disabling for now.
-            return gitHubClient.Repository.GetAllBranches(owner, repo);
-#pragma warning restore
+            return gitHubClient.Repository.Branch.GetAll(owner, repo);
         }
 
         public IObservable<Repository> GetRepository(string owner, string repo)

submodules/octokit.net

@@ -237,7 +237,7 @@ private static Repository CreateRepository(int id, bool hasWiki)
     {
         return new Repository("", "", "", "", "", "", "",
             id, new User(), "", "", "", "", "", false, false, 0, 0, "",
-            0, null, DateTimeOffset.Now, DateTimeOffset.Now, new RepositoryPermissions(), null, null, false,
+            0, null, DateTimeOffset.Now, DateTimeOffset.Now, new RepositoryPermissions(), null, null, null, false,
             hasWiki, false, false, 0, 0, null, null, null);
     }
 }

test/UnitTests/GitHub.Api/SimpleApiClientTests.cs

@@ -88,8 +88,8 @@ public async Task CanRetrieveAndCacheLicenses()
         {
             var data = new[]
             {
-                new LicenseMetadata("mit", "MIT", new Uri("https://github.com/")),
-                new LicenseMetadata("apache", "Apache", new Uri("https://github.com/"))
+                new LicenseMetadata("mit", "MIT", "foo", "https://github.com/", false),
+                new LicenseMetadata("apache", "Apache", "foo", "https://github.com/", false)
             };
 
             var apiClient = Substitute.For<IApiClient>();

test/UnitTests/GitHub.App/Models/ModelServiceTests.cs

@@ -27,7 +27,7 @@ public virtual void OnExit()
     protected static User CreateOctokitUser(string login = "login", string url = "https://url")
     {
         return new User("https://url", "bio", "blog", 1, "GitHub",
-            DateTimeOffset.UtcNow, 0, "email", 100, 100, true, url,
+            DateTimeOffset.UtcNow, DateTimeOffset.UtcNow, 0, "email", 100, 100, true, url,
             10, 42, "location", login, "name", 1, new Plan(),
             1, 1, 1, "https://url", new RepositoryPermissions(true, true, true),
             false, null, null);
@@ -46,7 +46,7 @@ protected static Repository CreateRepository(string owner, string name, string d
             id, CreateOctokitUser(owner),
             name, "fullname", "description", notCloneUrl, "c#", false, parent != null, 0, 0, "master",
             0, DateTimeOffset.UtcNow, DateTimeOffset.UtcNow, DateTimeOffset.UtcNow,
-            new RepositoryPermissions(), parent, null, true, false, false, false, 0, 0, null, null, null);
+            new RepositoryPermissions(), parent, null, null, true, false, false, false, 0, 0, null, null, null);
     }
 
     protected static PullRequest CreatePullRequest(User user, int id, ItemState state, string title,
@@ -58,18 +58,18 @@ protected static PullRequest CreatePullRequest(User user, int id, ItemState stat
             1, user, "Repo", "Repo", string.Empty, string.Empty, string.Empty,
             false, false, 0, 0, "master",
             0, null, createdAt, updatedAt,
-            null, null, null,
+            null, null, null, null,
             false, false, false,
             false, 0, 0,
             null, null, null);
-        return new PullRequest(0, uri, uri, uri, uri, uri, uri,
+        return new PullRequest(0, uris, uris, uris, uris, uris, uris,
             id, state, title, "", createdAt, updatedAt,
             null, null,
             new GitReference(uri.ToString(), "foo:bar", "bar", "123", user, repo),
             new GitReference(uri.ToString(), "foo:baz", "baz", "123", user, repo),
-            user, null, null, false, null,
-            commentCount, reviewCommentCount, 0, 0, 0, 0,
-            null, false);
+            user, null, null, false, null, null, null,
+            commentCount, 0, 0, 0, 0,
+            null, false, null);
     }
 
     protected class TempDirectory : IDisposable