Revert "Added a new overload of CreateMethodCall that accepts Expression<Func<TObject, IEnumerable<TValue>>> selector"

This reverts commit 3923872.

Author: MichaelDenwood

Octokit.GraphQL.Core/Core/Builders/QueryEntityBuilders.cs

@@ -1,10 +1,7 @@
 using System;
-using System.Collections;
 using System.Collections.Generic;
-using System.Linq;
 using System.Linq.Expressions;
 using System.Reflection;
-using Octokit.GraphQL.Core.Utilities;
 
 namespace Octokit.GraphQL.Core.Builders
 {
@@ -54,26 +51,6 @@ public static IQueryableValue<TValue> CreateMethodCall<TObject, TValue>(
                     arguments));
         }
 
-        public static IEnumerable<TValue> CreateMethodCall<TObject, TValue>(
-            this TObject o,
-            Expression<Func<TObject, IEnumerable<TValue>>> selector)
-            where TObject : IQueryableValue
-        {
-            var methodCall = (MethodCallExpression)selector.Body;
-            var arguments = new List<ConstantExpression>();
-
-            foreach (MemberExpression arg in methodCall.Arguments)
-            {
-                var expression = (ConstantExpression)arg.Expression;
-                var value = ((FieldInfo)arg.Member).GetValue(expression.Value);
-                arguments.Add(Expression.Constant(value, arg.Type));
-            }
-
-            return (IEnumerable<TValue>)(Expression.Call(Expression.Constant(o),
-                    methodCall.Method,
-                    arguments));
-        }
-
         public static TValue CreateMethodCall<TObject, TValue>(
             this TObject o,
             Expression<Func<TObject, TValue>> selector,

Octokit.GraphQL.Core/QueryableListExtensions.cs

@@ -70,8 +70,7 @@ public static IDictionary<TKey, TElement> ToDictionary<TSource, TKey, TElement>(
         [MethodId(nameof(ToListMethod))]
         public static List<TValue> ToList<TValue>(this IQueryableList<TValue> source)
         {
-            return source.ToList<TValue>();
-
+            throw new NotImplementedException();
         }
 
         public static ICompiledQuery<IEnumerable<T>> Compile<T>(this IQueryableList<T> expression)

Octokit.GraphQL.sln.DotSettings

@@ -5,7 +5,6 @@
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpKeepExistingMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpPlaceEmbeddedOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpRenamePlacementToArrangementMigration/@EntryIndexedValue">True</s:Boolean>
-	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ECSharpUseContinuousIndentInsideBracesMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EAddAccessorOwnerDeclarationBracesMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002ECSharpPlaceAttributeOnSameLineMigration/@EntryIndexedValue">True</s:Boolean>
 	<s:Boolean x:Key="/Default/Environment/SettingsMigration/IsMigratorApplied/=JetBrains_002EReSharper_002EPsi_002ECSharp_002ECodeStyle_002ESettingsUpgrade_002EMigrateBlankLinesAroundFieldToBlankLinesAroundProperty/@EntryIndexedValue">True</s:Boolean>

Octokit.GraphQL/Model/ActionExecutionCapabilitySetting.cs

@@ -0,0 +1,38 @@
+using System;
+using System.Runtime.Serialization;
+using Newtonsoft.Json;
+using Newtonsoft.Json.Converters;
+
+namespace Octokit.GraphQL.Model
+{
+    /// <summary>
+    /// The possible capabilities for action executions setting.
+    /// </summary>
+    [JsonConverter(typeof(StringEnumConverter))]
+    public enum ActionExecutionCapabilitySetting
+    {
+        /// <summary>
+        /// All action executions are disabled.
+        /// </summary>
+        [EnumMember(Value = "DISABLED")]
+        Disabled,
+
+        /// <summary>
+        /// All action executions are enabled.
+        /// </summary>
+        [EnumMember(Value = "ALL_ACTIONS")]
+        AllActions,
+
+        /// <summary>
+        /// Only actions defined within the repo are allowed.
+        /// </summary>
+        [EnumMember(Value = "LOCAL_ACTIONS_ONLY")]
+        LocalActionsOnly,
+
+        /// <summary>
+        /// Organization administrators action execution capabilities.
+        /// </summary>
+        [EnumMember(Value = "NO_POLICY")]
+        NoPolicy,
+    }
+}

Octokit.GraphQL/Model/BaseRefChangedEvent.cs

@@ -25,28 +25,13 @@ internal BaseRefChangedEvent(Expression expression) : base(expression)
         /// </summary>
         public DateTimeOffset CreatedAt { get; }
 
-        /// <summary>
-        /// Identifies the name of the base ref for the pull request after it was changed.
-        /// </summary>
-        public string CurrentRefName { get; }
-
         /// <summary>
         /// Identifies the primary key from the database.
         /// </summary>
         public int? DatabaseId { get; }
 
         public ID Id { get; }
 
-        /// <summary>
-        /// Identifies the name of the base ref for the pull request before it was changed.
-        /// </summary>
-        public string PreviousRefName { get; }
-
-        /// <summary>
-        /// PullRequest referenced by event.
-        /// </summary>
-        public PullRequest PullRequest => this.CreateProperty(x => x.PullRequest, Octokit.GraphQL.Model.PullRequest.Create);
-
         internal static BaseRefChangedEvent Create(Expression expression)
         {
             return new BaseRefChangedEvent(expression);

Octokit.GraphQL/Model/CommentAuthorAssociation.cs

@@ -23,12 +23,6 @@ public enum CommentAuthorAssociation
         [EnumMember(Value = "OWNER")]
         Owner,
 
-        /// <summary>
-        /// Author is a placeholder for an unclaimed user.
-        /// </summary>
-        [EnumMember(Value = "MANNEQUIN")]
-        Mannequin,
-
         /// <summary>
         /// Author has been invited to collaborate on the repository.
         /// </summary>

Octokit.GraphQL/Model/CommentDeletedEvent.cs

@@ -30,11 +30,6 @@ internal CommentDeletedEvent(Expression expression) : base(expression)
         /// </summary>
         public int? DatabaseId { get; }
 
-        /// <summary>
-        /// The user who authored the deleted comment.
-        /// </summary>
-        public IActor DeletedCommentAuthor => this.CreateProperty(x => x.DeletedCommentAuthor, Octokit.GraphQL.Model.Internal.StubIActor.Create);
-
         public ID Id { get; }
 
         internal static CommentDeletedEvent Create(Expression expression)

Octokit.GraphQL/Model/Commit.cs

@@ -50,16 +50,6 @@ internal Commit(Expression expression) : base(expression)
         /// </summary>
         public DateTimeOffset AuthoredDate { get; }
 
-        /// <summary>
-        /// The list of authors for this commit based on the git author and the Co-authored-by
-        /// message trailer. The git author will always be first.
-        /// </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 GitActorConnection Authors(Arg<int>? first = null, Arg<string>? after = null, Arg<int>? last = null, Arg<string>? before = null) => this.CreateMethodCall(x => x.Authors(first, after, last, before), Octokit.GraphQL.Model.GitActorConnection.Create);
-
         /// <summary>
         /// Fetches `git blame` information.
         /// </summary>
@@ -131,12 +121,6 @@ internal Commit(Expression expression) : base(expression)
         /// <param name="orderBy">Ordering options for deployments returned from the connection.</param>
         public DeploymentConnection Deployments(Arg<int>? first = null, Arg<string>? after = null, Arg<int>? last = null, Arg<string>? before = null, Arg<IEnumerable<string>>? environments = null, Arg<DeploymentOrder>? orderBy = null) => this.CreateMethodCall(x => x.Deployments(first, after, last, before, environments, orderBy), Octokit.GraphQL.Model.DeploymentConnection.Create);
 
-        /// <summary>
-        /// The tree entry representing the file located at the given path.
-        /// </summary>
-        /// <param name="path">The path for the file</param>
-        public TreeEntry File(Arg<string> path) => this.CreateMethodCall(x => x.File(path), Octokit.GraphQL.Model.TreeEntry.Create);
-
         /// <summary>
         /// The linear commit history starting from (and including) this commit, in the same order as `git log`.
         /// </summary>

Octokit.GraphQL/Model/CreateBranchProtectionRuleInput.cs

@@ -33,21 +33,6 @@ public class CreateBranchProtectionRuleInput
         /// </summary>
         public bool? RequiresCommitSignatures { get; set; }
 
-        /// <summary>
-        /// Are merge commits prohibited from being pushed to this branch.
-        /// </summary>
-        public bool? RequiresLinearHistory { get; set; }
-
-        /// <summary>
-        /// Are force pushes allowed on this branch.
-        /// </summary>
-        public bool? AllowsForcePushes { get; set; }
-
-        /// <summary>
-        /// Can this branch be deleted.
-        /// </summary>
-        public bool? AllowsDeletions { get; set; }
-
         /// <summary>
         /// Can admins overwrite branch protection.
         /// </summary>

Octokit.GraphQL/Model/CreateIssueInput.cs

@@ -43,11 +43,6 @@ public class CreateIssueInput
         /// </summary>
         public IEnumerable<ID> ProjectIds { get; set; }
 
-        /// <summary>
-        /// The name of an issue template in the repository, assigns labels and assignees from the template to the issue
-        /// </summary>
-        public string IssueTemplate { get; set; }
-
         /// <summary>
         /// A unique identifier for the client performing the mutation.
         /// </summary>