Merge pull request #221 from octokit/fixes/206-remove-union-hack

Stop returning IQueryableList for unions

Author: jcansdale

Octokit.GraphQL.Core.Generation/InterfaceGenerator.cs

@@ -75,14 +75,6 @@ private static string GenerateField(FieldModel field)
                     GenerateListMethod(field, reduced) :
                     GenerateListField(field, reduced);
             }
-            else if (reduced.Kind == TypeKind.Union)
-            {
-                // HACK: Returning IEnumerable<object> for unions for now until we decide how to handle them.
-                reduced = TypeModel.List(reduced);
-                result += method ?
-                    GenerateListMethod(field, reduced) :
-                    GenerateListField(field, reduced);
-            }
             else
             {
                 result += method ?

Octokit.GraphQL/Model/AuditEntry.cs

@@ -21,7 +21,7 @@ public interface IAuditEntry : IQueryableValue<IAuditEntry>, IQueryableInterface
         /// <summary>
         /// The user who initiated the action
         /// </summary>
-        IQueryableList<AuditEntryActor> Actor { get; }
+        AuditEntryActor Actor { get; }
 
         /// <summary>
         /// The IP address of the actor
@@ -96,7 +96,7 @@ internal StubIAuditEntry(Expression expression) : base(expression)
 
         public string Action { get; }
 
-        public IQueryableList<AuditEntryActor> Actor => this.CreateProperty(x => x.Actor);
+        public AuditEntryActor Actor => this.CreateProperty(x => x.Actor, Octokit.GraphQL.Model.AuditEntryActor.Create);
 
         public string ActorIp { get; }
 

Octokit.GraphQL/Model/CheckConclusionState.cs

@@ -48,13 +48,13 @@ public enum CheckConclusionState
         Neutral,
 
         /// <summary>
-        /// The check suite or run was skipped. For internal use only.
+        /// The check suite or run was skipped by GitHub. Only GitHub can use this conclusion.
         /// </summary>
         [EnumMember(Value = "SKIPPED")]
         Skipped,
 
         /// <summary>
-        /// The check suite or run was marked stale. For internal use only.
+        /// The check suite or run was marked stale by GitHub. Only GitHub can use this conclusion.
         /// </summary>
         [EnumMember(Value = "STALE")]
         Stale,

Octokit.GraphQL/Model/Commit.cs

@@ -200,6 +200,20 @@ internal Commit(Expression expression) : base(expression)
         /// </summary>
         public Status Status => this.CreateProperty(x => x.Status, Octokit.GraphQL.Model.Status.Create);
 
+        /// <summary>
+        /// Check and Status rollup information for this commit.
+        /// </summary>
+        public StatusCheckRollup StatusCheckRollup => this.CreateProperty(x => x.StatusCheckRollup, Octokit.GraphQL.Model.StatusCheckRollup.Create);
+
+        /// <summary>
+        /// Returns a list of all submodules in this repository as of this Commit parsed from the .gitmodules file.
+        /// </summary>
+        /// <param name="first">Returns the first _n_ elements from the list.</param>
+        /// <param name="after">Returns the elements in the list that come after the specified cursor.</param>
+        /// <param name="last">Returns the last _n_ elements from the list.</param>
+        /// <param name="before">Returns the elements in the list that come before the specified cursor.</param>
+        public SubmoduleConnection Submodules(Arg<int>? first = null, Arg<string>? after = null, Arg<int>? last = null, Arg<string>? before = null) => this.CreateMethodCall(x => x.Submodules(first, after, last, before), Octokit.GraphQL.Model.SubmoduleConnection.Create);
+
         /// <summary>
         /// Returns a URL to download a tarball archive for a repository.
         /// Note: For private repositories, these links are temporary and expire after five minutes.

Octokit.GraphQL/Model/CreateIpAllowListEntryInput.cs

@@ -0,0 +1,36 @@
+namespace Octokit.GraphQL.Model
+{
+    using System;
+    using System.Collections.Generic;
+
+    /// <summary>
+    /// Autogenerated input type of CreateIpAllowListEntry
+    /// </summary>
+    public class CreateIpAllowListEntryInput
+    {
+        /// <summary>
+        /// The ID of the owner for which to create the new IP allow list entry.
+        /// </summary>
+        public ID OwnerId { get; set; }
+
+        /// <summary>
+        /// An IP address or range of addresses in CIDR notation.
+        /// </summary>
+        public string AllowListValue { get; set; }
+
+        /// <summary>
+        /// An optional name for the IP allow list entry.
+        /// </summary>
+        public string Name { get; set; }
+
+        /// <summary>
+        /// Whether the IP allow list entry is active when an IP allow list is enabled.
+        /// </summary>
+        public bool IsActive { get; set; }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; set; }
+    }
+}

Octokit.GraphQL/Model/CreateIpAllowListEntryPayload.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 CreateIpAllowListEntry
+    /// </summary>
+    public class CreateIpAllowListEntryPayload : QueryableValue<CreateIpAllowListEntryPayload>
+    {
+        internal CreateIpAllowListEntryPayload(Expression expression) : base(expression)
+        {
+        }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; }
+
+        /// <summary>
+        /// The IP allow list entry that was created.
+        /// </summary>
+        public IpAllowListEntry IpAllowListEntry => this.CreateProperty(x => x.IpAllowListEntry, Octokit.GraphQL.Model.IpAllowListEntry.Create);
+
+        internal static CreateIpAllowListEntryPayload Create(Expression expression)
+        {
+            return new CreateIpAllowListEntryPayload(expression);
+        }
+    }
+}

Octokit.GraphQL/Model/CreatePullRequestInput.cs

@@ -41,6 +41,11 @@ public class CreatePullRequestInput
         /// </summary>
         public bool? MaintainerCanModify { get; set; }
 
+        /// <summary>
+        /// Indicates whether this pull request should be a draft.
+        /// </summary>
+        public bool? Draft { get; set; }
+
         /// <summary>
         /// A unique identifier for the client performing the mutation.
         /// </summary>

Octokit.GraphQL/Model/DeleteDeploymentInput.cs

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

Octokit.GraphQL/Model/DeleteDeploymentPayload.cs

@@ -0,0 +1,28 @@
+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 DeleteDeployment
+    /// </summary>
+    public class DeleteDeploymentPayload : QueryableValue<DeleteDeploymentPayload>
+    {
+        internal DeleteDeploymentPayload(Expression expression) : base(expression)
+        {
+        }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; }
+
+        internal static DeleteDeploymentPayload Create(Expression expression)
+        {
+            return new DeleteDeploymentPayload(expression);
+        }
+    }
+}

Octokit.GraphQL/Model/DeleteIpAllowListEntryInput.cs

@@ -0,0 +1,21 @@
+namespace Octokit.GraphQL.Model
+{
+    using System;
+    using System.Collections.Generic;
+
+    /// <summary>
+    /// Autogenerated input type of DeleteIpAllowListEntry
+    /// </summary>
+    public class DeleteIpAllowListEntryInput
+    {
+        /// <summary>
+        /// The ID of the IP allow list entry to delete.
+        /// </summary>
+        public ID IpAllowListEntryId { get; set; }
+
+        /// <summary>
+        /// A unique identifier for the client performing the mutation.
+        /// </summary>
+        public string ClientMutationId { get; set; }
+    }
+}