Correct bunch of spelling mistakes

resources/benchmark.js

@@ -21,7 +21,7 @@ const { sampleModule } = require('./benchmark-fork');
 const NS_PER_SEC = 1e9;
 const LOCAL = 'local';
 
-// The maximum time in secounds a benchmark is allowed to run before finishing.
+// The maximum time in seconds a benchmark is allowed to run before finishing.
 const maxTime = 5;
 // The minimum sample size required to perform statistical analysis.
 const minSamples = 5;
@@ -30,7 +30,7 @@ function LOCAL_DIR(...paths) {
   return path.join(__dirname, '..', ...paths);
 }
 
-// Build a benchmarkable environment for the given revision
+// Build a benchmark-friendly environment for the given revision
 // and returns path to its 'dist' directory.
 function prepareRevision(revision) {
   console.log(`🍳  Preparing ${revision}...`);

resources/build.js

@@ -50,8 +50,8 @@ function showStats() {
 
   for (const filepath of readdirRecursive('./dist')) {
     const name = filepath.split(path.sep).pop();
-    const [base, ...splitedExt] = name.split('.');
-    const ext = splitedExt.join('.');
+    const [base, ...splitExt] = name.split('.');
+    const ext = splitExt.join('.');
 
     const filetype = ext ? '*.' + ext : base;
     fileTypes[filetype] = fileTypes[filetype] || { filepaths: [], size: 0 };

resources/check-cover.js

@@ -36,8 +36,8 @@ async function getFullCoverage() {
       .filter(filepath => filepath.endsWith('.js'))
       .map(filepath => path.join('src/', filepath));
 
-    await Promise.all(files.map(getCoverage)).then(covarageObjs => {
-      for (const coverage of covarageObjs) {
+    await Promise.all(files.map(getCoverage)).then(coverages => {
+      for (const coverage of coverages) {
         fullCoverage[coverage.path] = coverage;
       }
     });

resources/check-cycles.js

@@ -0,0 +1,62 @@
+// @noflow
+
+'use strict';
+
+const os = require('os');
+const fs = require('fs');
+const path = require('path');
+const assert = require('assert');
+const { exec } = require('./utils');
+
+const tmpDir = fs.mkdtempSync(path.join(os.tmpdir(), 'flow-dep-graph'));
+const tmpFile = path.join(tmpDir, 'out.dot');
+
+exec(`yarn flow graph dep-graph --quiet --strip-root --out ${tmpFile}`);
+const dot = fs.readFileSync(tmpFile, 'utf-8');
+assert(dot.startsWith('digraph {\n') && dot.endsWith('\n}'));
+const dotLines = dot.split('\n').slice(1, -1);
+
+let depGraph = [];
+for (const line of dotLines) {
+  const [, from, to] = line.trim().match(/^"(.*?)" -> "(.*?)"$/);
+  assert(from && to);
+  depGraph.push([from, to]);
+}
+
+for (const [from, to] of depGraph) {
+  if (
+    path.basename(to) === 'index.js' &&
+    !path.dirname(to).endsWith('__fixtures__') &&
+    path.basename(from) !== 'index.js'
+  ) {
+    console.log(from);
+  }
+}
+
+let removedEdges;
+do {
+  removedEdges = 0;
+  const fromFiles = new Set();
+  const toFiles = new Set();
+
+  for (const [from, to] of depGraph) {
+    fromFiles.add(from);
+    toFiles.add(to);
+  }
+
+  console.log(depGraph.length);
+  // eslint-disable-next-line no-loop-func
+  depGraph = depGraph.filter(([from, to]) => {
+    if (!toFiles.has(from) || !fromFiles.has(to)) {
+      ++removedEdges;
+      return false;
+    }
+    return true;
+  });
+} while (removedEdges > 0);
+
+console.log('digraph {');
+for (const [from, to] of depGraph) {
+  console.log(`  "${from}" -> "${to}"`);
+}
+console.log('}');

resources/gen-changelog.js

@@ -51,7 +51,7 @@ const repoURLMatch = /https:\/\/github.com\/([^/]+)\/([^/]+).git/.exec(
   packageJSON.repository.url,
 );
 if (repoURLMatch == null) {
-  console.error('Cannot extract organisation and repo name from repo URL!');
+  console.error('Cannot extract organization and repo name from repo URL!');
   process.exit(1);
 }
 const [, githubOrg, githubRepo] = repoURLMatch;
@@ -80,7 +80,7 @@ function getChangeLog() {
 
 function genChangeLog(tag, date, allPRs) {
   const byLabel = {};
-  const commitersByLogin = {};
+  const committersByLogin = {};
 
   for (const pr of allPRs) {
     const labels = pr.labels.nodes
@@ -102,7 +102,7 @@ function genChangeLog(tag, date, allPRs) {
     }
     byLabel[label] = byLabel[label] || [];
     byLabel[label].push(pr);
-    commitersByLogin[pr.author.login] = pr.author;
+    committersByLogin[pr.author.login] = pr.author;
   }
 
   let changelog = `## ${tag || 'Unreleased'} (${date})\n`;
@@ -128,12 +128,12 @@ function genChangeLog(tag, date, allPRs) {
     }
   }
 
-  const commiters = Object.values(commitersByLogin).sort((a, b) =>
+  const committers = Object.values(committersByLogin).sort((a, b) =>
     (a.name || a.login).localeCompare(b.name || b.login),
   );
-  changelog += `\n#### Committers: ${commiters.length}\n`;
-  for (const commiter of commiters) {
-    changelog += `* ${commiter.name}([@${commiter.login}](${commiter.url}))\n`;
+  changelog += `\n#### Committers: ${committers.length}\n`;
+  for (const committer of committers) {
+    changelog += `* ${committer.name}([@${committer.login}](${committer.url}))\n`;
   }
 
   return changelog;

src/__tests__/starWarsData.js

@@ -116,7 +116,7 @@ export type Droid = {|
  * Helper function to get a character by ID.
  */
 function getCharacter(id) {
-  // Returning a promise just to illustrate GraphQL.js's support.
+  // Returning a promise just to illustrate that GraphQL.js supports it.
   return Promise.resolve(humanData[id] || droidData[id]);
 }
 

src/__tests__/starWarsQuery-test.js

@@ -108,7 +108,7 @@ describe('Star Wars Query Tests', () => {
             friends: [
               {
                 name: 'Luke Skywalker',
-                appearsIn: ['NEWHOPE', 'EMPIRE', 'JEDI'],
+                appearsIn: ['NEW_HOPE', 'EMPIRE', 'JEDI'],
                 friends: [
                   {
                     name: 'Han Solo',
@@ -126,7 +126,7 @@ describe('Star Wars Query Tests', () => {
               },
               {
                 name: 'Han Solo',
-                appearsIn: ['NEWHOPE', 'EMPIRE', 'JEDI'],
+                appearsIn: ['NEW_HOPE', 'EMPIRE', 'JEDI'],
                 friends: [
                   {
                     name: 'Luke Skywalker',
@@ -141,7 +141,7 @@ describe('Star Wars Query Tests', () => {
               },
               {
                 name: 'Leia Organa',
-                appearsIn: ['NEWHOPE', 'EMPIRE', 'JEDI'],
+                appearsIn: ['NEW_HOPE', 'EMPIRE', 'JEDI'],
                 friends: [
                   {
                     name: 'Luke Skywalker',

src/__tests__/starWarsSchema.js

@@ -27,7 +27,7 @@ import { getFriends, getHero, getHuman, getDroid } from './starWarsData';
  * Using our shorthand to describe type systems, the type system for our
  * Star Wars example is:
  *
- * enum Episode { NEWHOPE, EMPIRE, JEDI }
+ * enum Episode { NEW_HOPE, EMPIRE, JEDI }
  *
  * interface Character {
  *   id: String!
@@ -65,13 +65,13 @@ import { getFriends, getHero, getHuman, getDroid } from './starWarsData';
  * The original trilogy consists of three movies.
  *
  * This implements the following type system shorthand:
- *   enum Episode { NEWHOPE, EMPIRE, JEDI }
+ *   enum Episode { NEW_HOPE, EMPIRE, JEDI }
  */
 const episodeEnum = new GraphQLEnumType({
   name: 'Episode',
   description: 'One of the films in the Star Wars Trilogy',
   values: {
-    NEWHOPE: {
+    NEW_HOPE: {
       value: 4,
       description: 'Released in 1977.',
     },

src/execution/__tests__/executor-test.js

@@ -539,7 +539,7 @@ describe('Execute: Handles basic execution tasks', () => {
               }),
             ),
             resolve() {
-              return Promise.reject(new Error('Dangit'));
+              return Promise.reject(new Error('Oops'));
             },
           },
         },
@@ -561,7 +561,7 @@ describe('Execute: Handles basic execution tasks', () => {
       errors: [
         {
           locations: [{ column: 9, line: 3 }],
-          message: 'Dangit',
+          message: 'Oops',
           path: ['foods'],
         },
       ],

src/execution/__tests__/schema-test.js

@@ -144,7 +144,7 @@ describe('Execute: Handles execution with a complex schema', () => {
         title,
         body,
         hidden,
-        notdefined
+        notDefined
       }
     `);
 

src/execution/__tests__/variables-test.js

@@ -930,7 +930,7 @@ describe('Execute: Handles inputs', () => {
           fieldWithObjectInput(input: $input)
         }
       `;
-      const result = executeQuery(doc, { input: 'whoknows' });
+      const result = executeQuery(doc, { input: 'WhoKnows' });
 
       expect(result).to.deep.equal({
         errors: [

src/execution/execute.d.ts

@@ -177,7 +177,7 @@ export const defaultTypeResolver: GraphQLTypeResolver<any, any>;
 export const defaultFieldResolver: GraphQLFieldResolver<any, any>;
 
 /**
- * This method looks up the field on the given type defintion.
+ * This method looks up the field on the given type definition.
  * It has special casing for the two introspection fields, __schema
  * and __typename. __typename is special because it can always be
  * queried as a field, even in situations where no other fields

src/jsutils/__tests__/dedent-test.js

@@ -34,13 +34,13 @@ describe('dedent', () => {
 
   it('removes only the first level of indentation', () => {
     const output = dedent`
-            qux
-              quux
-                quuux
-                  quuuux
+            first
+              second
+                third
+                  fourth
     `;
     expect(output).to.equal(
-      ['qux', '  quux', '    quuux', '      quuuux', ''].join('\n'),
+      ['first', '  second', '    third', '      fourth', ''].join('\n'),
     );
   });
 

src/jsutils/__tests__/inspect-test.js

@@ -166,7 +166,7 @@ describe('inspect', () => {
     expect(inspect(customA)).to.equal('[Circular]');
   });
 
-  it('Use class names for the shortform of an object', () => {
+  it('Use class names for the short form of an object', () => {
     class Foo {
       foo: string;
 

src/language/__tests__/lexer-test.js

@@ -384,11 +384,11 @@ describe('Lexer', () => {
       value: 'contains " quote',
     });
 
-    expect(lexOne('"""contains \\""" triplequote"""')).to.contain({
+    expect(lexOne('"""contains \\""" triple quote"""')).to.contain({
       kind: TokenKind.BLOCK_STRING,
       start: 0,
-      end: 31,
-      value: 'contains """ triplequote',
+      end: 32,
+      value: 'contains """ triple quote',
     });
 
     expect(lexOne('"""multi\nline"""')).to.contain({

src/language/__tests__/parser-test.js

@@ -66,8 +66,8 @@ describe('Parser', () => {
       locations: [{ line: 1, column: 10 }],
     });
 
-    expectSyntaxError('notanoperation Foo { field }').to.deep.include({
-      message: 'Syntax Error: Unexpected Name "notanoperation".',
+    expectSyntaxError('notAnOperation Foo { field }').to.deep.include({
+      message: 'Syntax Error: Unexpected Name "notAnOperation".',
       locations: [{ line: 1, column: 1 }],
     });
 

src/language/__tests__/printLocation-test.js

@@ -11,7 +11,7 @@ import { printSourceLocation } from '../printLocation';
 describe('printSourceLocation', () => {
   it('prints minified documents', () => {
     const minifiedSource = new Source(
-      'query SomeMiniFiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String){someField(foo:$foo bar:$bar baz:SECOND_ERROR_HERE){fieldA fieldB{fieldC fieldD...on THIRD_ERROR_HERE}}}',
+      'query SomeMinifiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String){someField(foo:$foo bar:$bar baz:SECOND_ERROR_HERE){fieldA fieldB{fieldC fieldD...on THIRD_ERROR_HERE}}}',
     );
 
     const firstLocation = printSourceLocation(minifiedSource, {
@@ -20,7 +20,7 @@ describe('printSourceLocation', () => {
     });
     expect(firstLocation + '\n').to.equal(dedent`
       GraphQL request:1:53
-      1 | query SomeMiniFiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
+      1 | query SomeMinifiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
         |                                                     ^
         | {someField(foo:$foo bar:$bar baz:SECOND_ERROR_HERE){fieldA fieldB{fieldC fieldD.
     `);
@@ -31,7 +31,7 @@ describe('printSourceLocation', () => {
     });
     expect(secondLocation + '\n').to.equal(dedent`
       GraphQL request:1:114
-      1 | query SomeMiniFiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
+      1 | query SomeMinifiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
         | {someField(foo:$foo bar:$bar baz:SECOND_ERROR_HERE){fieldA fieldB{fieldC fieldD.
         |                                  ^
         | ..on THIRD_ERROR_HERE}}}
@@ -43,7 +43,7 @@ describe('printSourceLocation', () => {
     });
     expect(thirdLocation + '\n').to.equal(dedent`
       GraphQL request:1:166
-      1 | query SomeMiniFiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
+      1 | query SomeMinifiedQueryWithErrorInside($foo:String!=FIRST_ERROR_HERE$bar:String)
         | {someField(foo:$foo bar:$bar baz:SECOND_ERROR_HERE){fieldA fieldB{fieldC fieldD.
         | ..on THIRD_ERROR_HERE}}}
         |      ^

src/language/blockString.d.ts

@@ -1,6 +1,6 @@
 /**
  * Produces the value of a block string from its parsed raw value, similar to
- * Coffeescript's block string, Python's docstring trim or Ruby's strip_heredoc.
+ * CoffeeScript's block string, Python's docstring trim or Ruby's strip_heredoc.
  *
  * This implements the GraphQL spec's BlockStringValue() static algorithm.
  */

src/language/lexer.d.ts

@@ -42,7 +42,7 @@ export class Lexer {
 
   /**
    * Looks ahead and returns the next non-ignored token, but does not change
-   * the Lexer's state.
+   * the state of Lexer.
    */
   lookahead(): Token;
 }

src/language/lexer.js

@@ -59,7 +59,7 @@ export class Lexer {
 
   /**
    * Looks ahead and returns the next non-ignored token, but does not change
-   * the Lexer's state.
+   * the state of Lexer.
    */
   lookahead(): Token {
     let token = this.token;

src/language/printLocation.js

@@ -73,7 +73,8 @@ function printPrefixedLines(lines: $ReadOnlyArray<[string, string]>): string {
   const padLen = Math.max(...existingLines.map(([prefix]) => prefix.length));
   return existingLines
     .map(
-      ([prefix, line]) => lpad(padLen, prefix) + (line ? ' | ' + line : ' |'),
+      ([prefix, line]) =>
+        leftPad(padLen, prefix) + (line ? ' | ' + line : ' |'),
     )
     .join('\n');
 }
@@ -82,6 +83,6 @@ function whitespace(len: number): string {
   return Array(len + 1).join(' ');
 }
 
-function lpad(len: number, str: string): string {
+function leftPad(len: number, str: string): string {
   return whitespace(len - str.length) + str;
 }