Merge pull request rmosolgo#4980 from ydah/fix-typos

Fix some typos

Author: rmosolgo

CHANGELOG-pro.md

@@ -806,7 +806,7 @@
 
 ### Bug Fix
 
-- Pundit integration: use overriden `pundit_policy_class` for scoping and mutation authorization
+- Pundit integration: use overridden `pundit_policy_class` for scoping and mutation authorization
 
 ## 1.9.11 (20 Feb 2019)
 

CHANGELOG.md

@@ -324,7 +324,7 @@
 ### Bug fixes
 
 - Fix merging results after calling directives #4639 #4660
-- Visibility: don't reveal implementors of hidden abstract types #4589
+- Visibility: don't reveal implementers of hidden abstract types #4589
 - Bump required Ruby version to 2.7 since numbered block arguments are used  #4659
 - `hash_key:`: use the configured hash key when the underlying Hash has a default value Proc #4656
 
@@ -1583,7 +1583,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 
 ### New features
 
-- Add options to `implements(...)` and inteface type visibility #2791
+- Add options to `implements(...)` and interface type visibility #2791
 - Add `Query#fingerprint` for logging #2859
 - Add `--playground` option to install generator #2839
 - Support lazy-loaded objects from input object `loads:` #2834
@@ -1758,7 +1758,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 
 ### Breaking changes
 
-- `GraphQL::Schema::Resolver#initialize` accepts a new keyword argument, `field:`. If you have overriden this method, you'll have to add that keyword to your argument list (and pass it along to `super`.) #2605
+- `GraphQL::Schema::Resolver#initialize` accepts a new keyword argument, `field:`. If you have overridden this method, you'll have to add that keyword to your argument list (and pass it along to `super`.) #2605
 
 ### Deprecations
 
@@ -2158,7 +2158,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 ### Bug fixes
 
 - Argument default values include nested default values #1728
-- Clean up duplciate method defs #1739
+- Clean up duplicate method defs #1739
 
 ### New features
 
@@ -2682,7 +2682,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 - `GraphQL::Argument.define` builds re-usable arguments #948
 - `GraphQL::Subscriptions` provides hooks for subscription platforms #672
 - `GraphQL::Subscriptions::ActionCableSubscriptions` implements subscriptions over ActionCable #672
-- More runtime values are accessble from a `ctx` object #923 :
+- More runtime values are accessible from a `ctx` object #923 :
   - `ctx.parent` returns the `ctx` from the parent field
   - `ctx.object` returns the current `obj` for that field
   - `ctx.value` returns the resolved GraphQL value for that field
@@ -3201,7 +3201,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 
 - Absent variables aren't present in `args` #479
 - Fix grouped ActiveRecord relation with `last` only #476
-- `Schema#default_mask` & query `only:`/`except:` are combined, not overriden #485
+- `Schema#default_mask` & query `only:`/`except:` are combined, not overridden #485
 - Root types can be hidden with dynamic filters #480
 
 ## 1.4.0 (8 Jan 2017)
@@ -3383,7 +3383,7 @@ FieldExtension: pass extended values instead of originals to `after_resolve` #31
 
 ### Deprecations
 
-- `InternalRepresentation::Node#children` and `InternalRepresentation::Node#definitions` are deprecated due to the bug described below and the breaking change described above. Instead, use `InternalRepresentation::Node#typed_children` and `InternalRepresentation::Node#defininition`. #373
+- `InternalRepresentation::Node#children` and `InternalRepresentation::Node#definitions` are deprecated due to the bug described below and the breaking change described above. Instead, use `InternalRepresentation::Node#typed_children` and `InternalRepresentation::Node#definition`. #373
 
 ### New features
 

gemfiles/rails_master.gemfile

@@ -11,7 +11,7 @@ gem 'sqlite3'
 gem 'pg'
 gem "sequel"
 gem "evt"
-if RUBY_ENGINE == "ruby" # This doesn't work on truffle-ruby becuase there's no `IO::READABLE`
+if RUBY_ENGINE == "ruby" # This doesn't work on truffle-ruby because there's no `IO::READABLE`
   gem "libev_scheduler"
 end
 gem "async"

guides/authorization/visibility.md

@@ -3,7 +3,7 @@ layout: guide
 search: true
 section: Authorization
 title: Visibility
-desc: Programatically hide parts of the GraphQL schema from some users.
+desc: Programmatically hide parts of the GraphQL schema from some users.
 index: 1
 redirect_from:
 - /schema/limiting_visibility

guides/dataloader/adopting.md

@@ -26,7 +26,7 @@ In this example, a single object is batch-loaded to satisfy a GraphQL field.
   record_promise = Loaders::Record.load(1)
   ```
 
-  Then, under the hood, GraphQL-Ruby manages the promise (using its `lazy_resolve` feature, upstreamed from GraphQL-Batch many years ago). GraphQL-Ruby will call `.sync` on it when no futher execution is possible; `promise.rb` implements `Promise#sync` to execute the pending work.
+  Then, under the hood, GraphQL-Ruby manages the promise (using its `lazy_resolve` feature, upstreamed from GraphQL-Batch many years ago). GraphQL-Ruby will call `.sync` on it when no further execution is possible; `promise.rb` implements `Promise#sync` to execute the pending work.
 
 - With __GraphQL::Dataloader__, you get a source, then call `.load` on it, which may pause the current Fiber, but it returns the requested object.
 
@@ -59,7 +59,7 @@ In this example, one object is loaded, then another object is loaded _based on_
 
 ## Comparison: Fetching objects concurrently (independent)
 
-Sometimes, you need multiple _independent_ records to perform a calcuation. Each record is loaded, then they're combined in some bit of work.
+Sometimes, you need multiple _independent_ records to perform a calculation. Each record is loaded, then they're combined in some bit of work.
 
 - With __GraphQL-Batch__, `Promise.all(...)` is used to to wait for several pending loads:
 

guides/fields/introduction.md

@@ -213,7 +213,7 @@ A few `extras` are available:
 - `owner` (the type that this field belongs to)
 - `lookahead` (see {% internal_link "Lookahead", "/queries/lookahead" %})
 - `execution_errors`, whose `#add(err_or_msg)` method should be used for adding errors
-- `argument_details` (Intepreter only), an instance of {{ "GraphQL::Execution::Interpreter::Arguments" | api_doc }} with argument metadata
+- `argument_details` (Interpreter only), an instance of {{ "GraphQL::Execution::Interpreter::Arguments" | api_doc }} with argument metadata
 - `parent` (the previous `object` in the query)
 - Custom extras, see below
 

guides/limiters/active_operations.md

@@ -107,7 +107,7 @@ The hooks are:
 
 ## Instrumentation
 
-While the limiter is installed, it adds some information to the query context about its operation. It can be acccessed at `context[:active_operation_limiter]`:
+While the limiter is installed, it adds some information to the query context about its operation. It can be accessed at `context[:active_operation_limiter]`:
 
 ```ruby
 result = MySchema.execute(...)

guides/limiters/runtime.md

@@ -108,7 +108,7 @@ The hooks are:
 
 ## Instrumentation
 
-While the limiter is installed, it adds some information to the query context about its operation. It can be acccessed at `context[:runtime_limiter]`:
+While the limiter is installed, it adds some information to the query context about its operation. It can be accessed at `context[:runtime_limiter]`:
 
 
 ```ruby

guides/pagination/cursors.md

@@ -10,7 +10,7 @@ index: 4
 
 Connections use _cursors_ to advance through paginated lists. A cursor is an opaque string that indicates a specific point in this.
 
-Here, _opaque_ means that the string has no meaning except its value. cursors shouldn't be decoded, reverse-engineered, or generated ad-hoc. The only guarantee of a cursor is that, after you retrieve one, you can use it to request subsequent or preceeding items in the list.
+Here, _opaque_ means that the string has no meaning except its value. cursors shouldn't be decoded, reverse-engineered, or generated ad-hoc. The only guarantee of a cursor is that, after you retrieve one, you can use it to request subsequent or preceding items in the list.
 
 Although cursors can be tricky, they were chosen for Relay-style connections because they can be implemented in stable and high-performing ways.
 

guides/queries/ast_analysis.md

@@ -31,7 +31,7 @@ Pass the **class** (and not an _instance_) of your analyzer. The analysis engine
 Analyzers respond to methods similar to AST visitors. They're named like `on_enter_#{ast_node}` and `on_leave_#{ast_node}`. Methods are called with three arguments:
 
 - `node`: The current AST node (being entered or left)
-- `parent`: The AST node which preceeds this one in the tree
+- `parent`: The AST node which precedes this one in the tree
 - `visitor`: A {{ "GraphQL::Analysis::AST::Visitor" | api_doc }} which is managing this analysis run
 
 For example:

guides/schema/dynamic_types.md

@@ -207,7 +207,7 @@ end
 
 You can also use different types for each query. A few behaviors depend on the methods defined above:
 
-- If a type is not used as a return type, an argument type, or as a member of a union or implementor of an interface, it will be hidden
+- If a type is not used as a return type, an argument type, or as a member of a union or implementer of an interface, it will be hidden
 - If an interface or union has members, it will be hidden
 - If a field's return type is hidden, the field will be hidden
 - If an argument's input type is hidden, the argument will be hidden

guides/subscriptions/multi_tenant.md

@@ -42,7 +42,7 @@ BudgetSchema.subscriptions.trigger(:budget_was_approved, {}, { ... }, scope: "ac
 Alternatively, `subscription_scope` might name something that _belongs_ to the tenant:
 
 ```ruby
-class BudgetWasApproved < GraphQL::Schema::Subcription
+class BudgetWasApproved < GraphQL::Schema::Subscription
   subscription_scope :project_id # This would work with `context[:project_id] = 1234`
 end
 

guides/type_definitions/interfaces.md

@@ -177,7 +177,7 @@ end
 
 The type definition DSL uses this mechanism, too, so you can override those methods here also.
 
-Note: Under the hood, `definition_methods` causes a module to be `extend`ed by the Inteface. Any calls to `extend` or `implement` may override methods from `definition_methods`.
+Note: Under the hood, `definition_methods` causes a module to be `extend`ed by the interface. Any calls to `extend` or `implement` may override methods from `definition_methods`.
 
 ### Resolve Type
 

javascript_client/src/subscriptions/__tests__/createAblyHandlerTest.ts

@@ -408,7 +408,7 @@ describe("createAblyHandler", () => {
       }
       await Promise.all(disposals)
 
-      // 201st subscription - should work now that previous 200 subscriptions have been diposed
+      // 201st subscription - should work now that previous 200 subscriptions have been disposed
       const { dispose } = ablyHandler(
         operation,
         variables,

javascript_client/src/subscriptions/registry.ts

@@ -3,7 +3,7 @@ interface ApolloSubscription {
 }
 
 // State management for subscriptions.
-// Used to add subscriptions to an Apollo network intrface.
+// Used to add subscriptions to an Apollo network interface.
 class ApolloSubscriptionRegistry {
   // Apollo expects unique ids to reference each subscription,
   // here's a simple incrementing ID generator which starts at 1

javascript_client/src/sync/index.ts

@@ -171,7 +171,7 @@ function sync(options: SyncOptions) {
             })
 
             var failed = responseData.failed.length
-            // These might get overriden for status output
+            // These might get overridden for status output
             var notModified = responseData.not_modified.length
             var added = responseData.added.length
             if (failed) {

lib/generators/graphql/install/mutation_root_generator.rb

@@ -9,7 +9,7 @@ module Install
       class MutationRootGenerator < Rails::Generators::Base
         include Core
 
-        desc "Create mutation base type, mutation root tipe, and adds the latter to the schema"
+        desc "Create mutation base type, mutation root type, and adds the latter to the schema"
         source_root File.expand_path('../templates', __FILE__)
 
         class_option :schema,
@@ -31,4 +31,4 @@ def generate
       end
     end
   end
-end 
+end 

lib/graphql/analysis/ast/query_complexity.rb

@@ -12,7 +12,7 @@ def initialize(query)
           @complexities_on_type_by_query = {}
         end
 
-        # Overide this method to use the complexity result
+        # Override this method to use the complexity result
         def result
           max_possible_complexity
         end

lib/graphql/dataloader/null_dataloader.rb

@@ -8,7 +8,7 @@ class Dataloader
     # simple internal code while adding the option to add Dataloader.
     class NullDataloader < Dataloader
       # These are all no-ops because code was
-      # executed sychronously.
+      # executed synchronously.
       def run; end
       def run_isolated; yield; end
       def yield

lib/graphql/execution/interpreter/runtime.rb

@@ -130,7 +130,7 @@ def gather_selections(owner_object, owner_type, selections, selections_to_run =
               selections = selections_by_name[response_key]
               # if there was already a selection of this field,
               # use an array to hold all selections,
-              # otherise, use the single node to represent the selection
+              # otherwise, use the single node to represent the selection
               if selections
                 # This field was already selected at least once,
                 # add this node to the list of selections

lib/graphql/rubocop/graphql/base_cop.rb

@@ -9,7 +9,7 @@ class BaseCop < RuboCop::Cop::Base
 
         # Return the source of `send_node`, but without the keyword argument represented by `pair_node`
         def source_without_keyword_argument(send_node, pair_node)
-          # work back to the preceeding comma
+          # work back to the preceding comma
           first_pos = pair_node.location.expression.begin_pos
           end_pos = pair_node.location.expression.end_pos
           node_source = send_node.source_range.source

lib/graphql/schema.rb

@@ -187,7 +187,7 @@ def trace_class_for(mode, build: false)
       # {default_trace_mode} is used when no `trace_mode: ...` is requested.
       #
       # When a `trace_class` is added this way, it will _not_ receive other modules added with `trace_with(...)`
-      # unless `trace_mode` is explicitly given. (This class will not recieve any default trace modules.)
+      # unless `trace_mode` is explicitly given. (This class will not receive any default trace modules.)
       #
       # Subclasses of the schema will use `trace_class` as a base class for this mode and those
       # subclass also will _not_ receive default tracing modules.

lib/graphql/schema/list.rb

@@ -52,7 +52,7 @@ def validate_non_null_input(value, ctx, max_errors: nil)
           unless item_result.valid?
             if max_errors
               if max_errors == 0
-                add_max_errros_reached_message(result)
+                add_max_errors_reached_message(result)
                 break
               end
 
@@ -76,7 +76,7 @@ def ensure_array(value)
         end
       end
 
-      def add_max_errros_reached_message(result)
+      def add_max_errors_reached_message(result)
         message = "Too many errors processing list variable, max validation error limit reached. Execution aborted"
         item_result = GraphQL::Query::InputValidationResult.from_problem(message)
         result.merge_result!(nil, item_result)

lib/graphql/schema/member/has_arguments.rb

@@ -259,8 +259,8 @@ def argument_class(new_arg_class = nil)
         #
         # @param values [Hash<String, Object>]
         # @param context [GraphQL::Query::Context]
-        # @yield [Interpreter::Arguments, Execution::Lazy<Interpeter::Arguments>]
-        # @return [Interpreter::Arguments, Execution::Lazy<Interpeter::Arguments>]
+        # @yield [Interpreter::Arguments, Execution::Lazy<Interpreter::Arguments>]
+        # @return [Interpreter::Arguments, Execution::Lazy<Interpreter::Arguments>]
         def coerce_arguments(parent_object, values, context, &block)
           # Cache this hash to avoid re-merging it
           arg_defns = context.warden.arguments(self)

lib/graphql/schema/member/has_directives.rb

@@ -91,7 +91,7 @@ def get_directives(schema_member, directives, directives_method)
           private
 
           # Modify `target` by adding items from `dirs` such that:
-          # - Any name conflict is overriden by the incoming member of `dirs`
+          # - Any name conflict is overridden by the incoming member of `dirs`
           # - Any other member of `dirs` is appended
           # @param target [Array<GraphQL::Schema::Directive>]
           # @param dirs [Array<GraphQL::Schema::Directive>]

lib/graphql/schema/member/relay_shortcuts.rb

@@ -76,7 +76,7 @@ def configured_edge_type_class
 
         private
 
-        # If one of thse values is accessed, initialize all the instance variables to retain
+        # If one of these values is accessed, initialize all the instance variables to retain
         # a consistent object shape.
         def initialize_relay_metadata
           if !defined?(@connection_type)

lib/graphql/schema/warden.rb

@@ -295,7 +295,7 @@ def visible_and_reachable_type?(type_defn)
               true
             else
               if @context.respond_to?(:logger) && (logger = @context.logger)
-                logger.debug { "Interface `#{type_defn.graphql_name}` hidden because it has no visible implementors" }
+                logger.debug { "Interface `#{type_defn.graphql_name}` hidden because it has no visible implementers" }
               end
               false
             end

lib/graphql/static_validation/rules/fields_will_merge.rb

@@ -396,7 +396,7 @@ def compared_fragments_key(frag1, frag2, exclusive)
       end
 
       # Given two list of parents, find out if they are mutually exclusive
-      # In this context, `parents` represends the "self scope" of the field,
+      # In this context, `parents` represents the "self scope" of the field,
       # what types may be found at this point in the query.
       def mutually_exclusive?(parents1, parents2)
         if parents1.empty? || parents2.empty?

lib/graphql/subscriptions/action_cable_subscriptions.rb

@@ -107,7 +107,7 @@ def initialize(serializer: Serialize, namespace: '', action_cable: ActionCable,
         when 2
           true
         else
-          raise ArgumentError, "#{@serializer} must repond to `.load` accepting one or two arguments"
+          raise ArgumentError, "#{@serializer} must respond to `.load` accepting one or two arguments"
         end
         @transmit_ns = namespace
         super

lib/graphql/type_kinds.rb

@@ -16,7 +16,7 @@ def initialize(name, abstract: false, leaf: false, fields: false, wraps: false,
         @description = description
       end
 
-      # Does this TypeKind have multiple possible implementors?
+      # Does this TypeKind have multiple possible implementers?
       # @deprecated Use `abstract?` instead of `resolves?`.
       def resolves?;  @abstract;  end
       # Is this TypeKind abstract?

spec/graphql/analysis/ast/query_complexity_spec.rb

@@ -387,7 +387,7 @@
     end
   end
 
-  describe "calucation complexity for a multiplex" do
+  describe "calculation complexity for a multiplex" do
     let(:query_string) {%|
       query cheeses {
         cheese(id: 1) {

spec/graphql/authorization_spec.rb

@@ -453,7 +453,7 @@ def auth_execute(*args, **kwargs)
       assert_equal "RelayObjectEdge", visible_res["data"]["hiddenEdge"]["__typename"]
     end
 
-    it "treats hidden enum values as non-existant, even in lists" do
+    it "treats hidden enum values as non-existent, even in lists" do
       hidden_res_1 = auth_execute <<-GRAPHQL, context: { hide: true }
       {
         landscapeFeature(enum: TAR_PIT)

spec/graphql/execution/interpreter_spec.rb

@@ -143,7 +143,7 @@ def interpreter_context_for(key)
         base_ctx_value = context[key]
         interpreter_ctx_value = context.namespace(:interpreter)[key]
         if base_ctx_value != interpreter_ctx_value
-          raise "Context mismatch for #{key} -> #{base_ctx_value} / intepreter: #{interpreter_ctx_value}"
+          raise "Context mismatch for #{key} -> #{base_ctx_value} / interpreter: #{interpreter_ctx_value}"
         else
           base_ctx_value
         end

spec/graphql/language/document_from_schema_definition_spec.rb

@@ -322,7 +322,7 @@ class Query < GraphQL::Schema::Object
       end
     end
 
-    describe "with a visiblity check" do
+    describe "with a visibility check" do
       let(:expected_idl) { <<-GRAPHQL
         type QueryType {
           foo: Foo

spec/graphql/language/parser_spec.rb

@@ -312,7 +312,7 @@
     assert_equal "b\\", document.definitions[0].selections[0].arguments[1].value
   end
 
-  it "parses backslases in non-last arguments" do
+  it "parses backslashes in non-last arguments" do
     document = subject.parse <<-GRAPHQL
       query {
         item(text: "b\\\\", otherText: "a") {