Merge branch 'main' into cleanup-projects

Author: kfcampbell

.github/workflows/add_to_octokit_project.yml

@@ -12,7 +12,7 @@ jobs:
     runs-on: ubuntu-latest
     continue-on-error: true
     steps:
-      - uses: actions/add-to-project@v0.4.0
+      - uses: actions/add-to-project@v0.5.0
         with:
           project-url: https://github.com/orgs/octokit/projects/10
           github-token: ${{ secrets.OCTOKITBOT_PROJECT_ACTION_TOKEN }}

.github/workflows/immediate-response.yml

@@ -6,23 +6,24 @@ on:
   issues:
     types:
       - opened
-  pull_request:
+  pull_request_target:
     types:
       - opened
 jobs:
   respond-to-issue:
+    if: ${{ github.actor != 'dependabot[bot]' && github.actor != 'renovate[bot]' && github.actor != 'githubactions[bot]' && github.actor != 'octokitbot' }}
     runs-on: ubuntu-latest
     steps:
       - name: Determine issue or PR number
         id: extract
         run: echo "NUMBER=${{ github.event.issue.number || github.event.pull_request.number }}" >> "$GITHUB_OUTPUT"
 
       - name: Respond to issue or PR
-        uses: peter-evans/create-or-update-comment@6534843181fc2aeb7f9f1cd3cd4a7b956cada2db
+        uses: peter-evans/create-or-update-comment@v3
         with:
           issue-number: ${{ steps.extract.outputs.NUMBER }}
           body: >
             👋 Hi! Thank you for this contribution! Just to let you know, our GitHub SDK team does a round of issue and PR reviews twice a week, every Monday and Friday!
             We have a [process in place](https://github.com/octokit/.github/blob/main/community/prioritization_response.md#overview) for prioritizing and responding to your input. 
             Because you are a part of this community please feel free to comment, add to, or pick up any issues/PRs that are labled with `Status: Up for grabs`.
-            You & others like you are the reason all of this works! So thank you & happy coding! 🚀
+            You & others like you are the reason all of this works! So thank you & happy coding! 🚀

Octokit.GraphQL.Core.UnitTests/QueryBuilderTests.cs

@@ -1264,6 +1264,26 @@ public void Repository_PullRequest_CheckRun_Aliased_Id()
             Assert.Equal(expected, actual, ignoreLineEndingDifferences: true);
         }
 
+        [Fact]
+        public void Repository_Parent_ConditionalExpression()
+        {
+            var expected = @"query {
+  repository(owner: ""foo"", name: ""bar"") {
+    parent {
+      name
+    }
+  }
+}";
+
+            var expression = new Query()
+                .Repository("foo", "bar")
+                .Select(repository => repository.Parent == null ? null : repository.Parent.Name);
+
+            var query = expression.Compile();
+
+            Assert.Equal(expected, query.ToString(2), ignoreLineEndingDifferences: true);
+        }
+
         class TestModelObject
         {
             public string StringField1;

Octokit.GraphQL.Core/Core/Builders/QueryBuilder.cs

@@ -1,4 +1,4 @@
-using System;
+using System;
 using System.Collections.Generic;
 using System.Collections.ObjectModel;
 using System.Linq;
@@ -204,7 +204,7 @@ protected override Expression VisitConditional(ConditionalExpression node)
             }
             else if (!falseNull)
             {
-                ifTrue = Expression.Constant(null, ifFalse.Type);
+                ifTrue = Expression.Constant(null, node.Type);
             }
 
             if (!falseNull)
@@ -213,14 +213,14 @@ protected override Expression VisitConditional(ConditionalExpression node)
             }
             else if (!trueNull)
             {
-                ifFalse = Expression.Constant(null, ifTrue.Type);
+                ifFalse = Expression.Constant(null, node.Type);
             }
 
             return Expression.Condition(
                 test,
                 ifTrue,
                 ifFalse,
-                !IsNullConstant(ifTrue) ? ifTrue.Type : ifFalse.Type);
+                node.Type);
         }
 
         protected override Expression VisitExtension(Expression node)

Octokit.GraphQL.UnitTests/ExpressionRewiterTests.cs

@@ -129,7 +129,7 @@ public void Repository_Details_With_Viewer()
         {
             var query = new Query()
                 .Select(x => x.RepositoryOwner("foo")
-                              .Repositories(30, null, null, null, null, null, null, null, null, null, null)
+                              .Repositories(30, null, null, null, null, null, null, null, null, null, null, null)
                               .Edges
                               .Select(y => y.Node)
                               .Select(z => new

Octokit.GraphQL.UnitTests/QueryBuilderTests.cs

@@ -120,7 +120,7 @@ public void RepositoryOwner_Repositories_Query_Viewer()
             var expression = new Query()
                 .Select(root => root
                     .RepositoryOwner("foo")
-                    .Repositories(30, null, null, null, null, null, null, null, null, null, null)
+                    .Repositories(30, null, null, null, null, null, null, null, null, null, null, null)
                     .Edges.Select(x => x.Node)
                     .Select(x => new
                     {
@@ -221,7 +221,7 @@ public void Repository_Details_With_Viewer()
 
             var expression = new Query()
                 .Select(x => x.RepositoryOwner("foo")
-                              .Repositories(30, null, null, null, null, null, null, null, null, null, null)
+                              .Repositories(30, null, null, null, null, null, null, null, null, null, null, null)
                               .Edges
                               .Select(y => y.Node)
                               .Select(y => new

Octokit.GraphQL.nuspec

@@ -2,7 +2,7 @@
 <package >
   <metadata>
     <id>Octokit.GraphQL</id>
-    <version>0.2.1-beta</version>
+    <version>0.3.0-beta</version>
     <authors>GitHub</authors>
     <owners>GitHub</owners>
     <repository type="git" url="https://github.com/octokit/octokit.graphql.net" />

Octokit.GraphQL/Model/AbortRepositoryMigrationInput.cs

@@ -0,0 +1,21 @@
+namespace Octokit.GraphQL.Model
+{
+    using System;
+    using System.Collections.Generic;
+
+    /// <summary>
+    /// Autogenerated input type of AbortRepositoryMigration
+    /// </summary>
+    public class AbortRepositoryMigrationInput
+    {
+        /// <summary>
+        /// The ID of the migration to be aborted.
+        /// </summary>
+        public ID MigrationId { get; set; }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; set; }
+    }
+}

Octokit.GraphQL/Model/AbortRepositoryMigrationPayload.cs

@@ -0,0 +1,33 @@
+namespace Octokit.GraphQL.Model
+{
+    using System;
+    using System.Collections.Generic;
+    using System.Linq.Expressions;
+    using Octokit.GraphQL.Core;
+    using Octokit.GraphQL.Core.Builders;
+
+    /// <summary>
+    /// Autogenerated return type of AbortRepositoryMigration
+    /// </summary>
+    public class AbortRepositoryMigrationPayload : QueryableValue<AbortRepositoryMigrationPayload>
+    {
+        internal AbortRepositoryMigrationPayload(Expression expression) : base(expression)
+        {
+        }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; }
+
+        /// <summary>
+        /// Did the operation succeed?
+        /// </summary>
+        public bool? Success { get; }
+
+        internal static AbortRepositoryMigrationPayload Create(Expression expression)
+        {
+            return new AbortRepositoryMigrationPayload(expression);
+        }
+    }
+}

Octokit.GraphQL/Model/AddPullRequestReviewThreadReplyInput.cs

@@ -0,0 +1,31 @@
+namespace Octokit.GraphQL.Model
+{
+    using System;
+    using System.Collections.Generic;
+
+    /// <summary>
+    /// Autogenerated input type of AddPullRequestReviewThreadReply
+    /// </summary>
+    public class AddPullRequestReviewThreadReplyInput
+    {
+        /// <summary>
+        /// The Node ID of the pending review to which the reply will belong.
+        /// </summary>
+        public ID? PullRequestReviewId { get; set; }
+
+        /// <summary>
+        /// The Node ID of the thread to which this reply is being written.
+        /// </summary>
+        public ID PullRequestReviewThreadId { get; set; }
+
+        /// <summary>
+        /// The text of the reply.
+        /// </summary>
+        public string Body { get; set; }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; set; }
+    }
+}