Spelling

docs_app/README.md

@@ -80,7 +80,7 @@ You also want to see those changes displayed properly in the doc viewer
 with a quick, edit/view cycle time.
 
 For this purpose, use the `npm run docs-watch` task, which watches for changes to source files and only
-re-processes the the files necessary to generate the docs that are related to the file that has changed.
+re-processes the files necessary to generate the docs that are related to the file that has changed.
 Since this task takes shortcuts, it is much faster (often less than 1 second) but it won't produce full
 fidelity content. For example, links to other docs and code examples may not render correctly. This is
 most particularly noticed in links to other docs and in the embedded examples, which may not always render

docs_app/content/6-to-7-change-summary.md

@@ -116,7 +116,7 @@ This document contains a detailed list of changes between RxJS 6.x and RxJS 7.x,
 
 #### Subscribable
 
-- `subscribe` will accept `Partial<Observer<T>>` now. All overloads with functions as arguments have been removed. This is because `Subscribable` is intended to map to the basic observable contract from the TC39 proposal and the the return type of a call to `[Symbol.observable]()`.
+- `subscribe` will accept `Partial<Observer<T>>` now. All overloads with functions as arguments have been removed. This is because `Subscribable` is intended to map to the basic observable contract from the TC39 proposal and the return type of a call to `[Symbol.observable]()`.
 
 #### SubscribableOrPromise
 

docs_app/content/deprecations/breaking-changes.md

@@ -24,7 +24,7 @@
 
 - The static `sortActions` method on `VirtualTimeScheduler` is no longer publicly exposed by our TS types.
 
-- `Notification.createNext(undefined)` will no longer return the exact same reference everytime.
+- `Notification.createNext(undefined)` will no longer return the exact same reference every time.
 
 - Type signatures tightened up around `Notification` and `dematerialize`, may uncover issues with invalid types passed to those operators.
 

docs_app/content/guide/installation.md

@@ -9,8 +9,8 @@ npm install rxjs
 ```
 
 By default, RxJS 7.x will provide different variants of the code based on the consumer:
-* When RxJS 7.x is used on Node.js regardless of whether it is consumed via `require` or `import`, CommonJS code targetting ES5 will be provided for execution.
-* When RxJS 7.4+ is used via a bundler targeting a browser (or other non-Node.js platform) ES module code targetting ES5 will be provided by default with the option to use ES2015 code.
+* When RxJS 7.x is used on Node.js regardless of whether it is consumed via `require` or `import`, CommonJS code targeting ES5 will be provided for execution.
+* When RxJS 7.4+ is used via a bundler targeting a browser (or other non-Node.js platform) ES module code targeting ES5 will be provided by default with the option to use ES2015 code.
 7.x versions prior to 7.4.0 will only provide ES5 code.
 
 If the target browsers for a project support ES2015+ or the bundle process supports down-leveling to ES5 then the bundler can optionally be configured to allow the ES2015 RxJS code to be used instead.

docs_app/content/guide/operators.md

@@ -282,7 +282,7 @@ function discardOddDoubleEven() {
 
 ### Creating new operators from scratch
 
-It is more complicated, but if you have to write an operator that cannot be made from a combination of existing operators (a rare occurrance), you can write an operator from scratch using the Observable constructor, like this:
+It is more complicated, but if you have to write an operator that cannot be made from a combination of existing operators (a rare occurrence), you can write an operator from scratch using the Observable constructor, like this:
 
 ```ts
 import { Observable, of } from 'rxjs';

docs_app/src/app/app.component.spec.ts

@@ -1011,7 +1011,7 @@ describe('AppComponent', () => {
         jasmine.clock().tick(1);  // triggers the HTTP response for the document
         const toolbar = fixture.debugElement.query(By.css('.app-toolbar'));
 
-        // Initially, `isTransitoning` is true.
+        // Initially, `isTransitioning` is true.
         expect(component.isTransitioning).toBe(true);
         expect(toolbar.classes['transitioning']).toBe(true);
 
@@ -1020,7 +1020,7 @@ describe('AppComponent', () => {
         expect(component.isTransitioning).toBe(false);
         expect(toolbar.classes['transitioning']).toBeFalsy();
 
-        // While a document is being rendered, `isTransitoning` is set to true.
+        // While a document is being rendered, `isTransitioning` is set to true.
         triggerDocViewerEvent('docReady');
         fixture.detectChanges();
         expect(component.isTransitioning).toBe(true);

docs_app/src/app/app.component.ts

@@ -135,7 +135,7 @@ export class AppComponent implements OnInit {
    *
    * * `page-...`: computed from the current document id (e.g. events, guide-security, tutorial-toh-pt2)
    * * `folder-...`: computed from the top level folder for an id (e.g. guide, tutorial, etc)
-   * * `view-...`: computef from the navigation view (e.g. SideNav, TopBar, etc)
+   * * `view-...`: computed from the navigation view (e.g. SideNav, TopBar, etc)
    */
   @HostBinding('class')
   hostClasses = '';
@@ -234,7 +234,7 @@ export class AppComponent implements OnInit {
     ).subscribe(([versionInfo, versions]) => {
       this.docVersions = [...versions];
 
-      // Find the current version - eithers title matches the current deployment mode
+      // Find the current version - either title matches the current deployment mode
       // or its title matches the major version of the current version info
       this.currentDocVersion = this.docVersions.find(
         (version) => version.title === this.deployment.mode || version.title === `v${versionInfo.major}`

docs_app/src/app/custom-elements/api/api-list.component.spec.ts

@@ -32,7 +32,7 @@ describe('ApiListComponent', () => {
    * Expectation Utility: Assert that filteredSections has the expected result for this test
    * @param itemTest - return true if the item passes the match test
    *
-   * Subscibes to `filteredSections` and performs expectation within subscription callback.
+   * Subscribes to `filteredSections` and performs expectation within subscription callback.
    */
   function expectFilteredResult(label: string, itemTest: (item: ApiItem) => boolean) {
     component.filteredSections.subscribe(filtered => {
@@ -96,7 +96,7 @@ describe('ApiListComponent', () => {
     });
 
     it('should have no sections and no items visible when there is no match', () => {
-      component.setQuery('fizbuzz');
+      component.setQuery('fizzbuzz');
       component.filteredSections.subscribe(filtered => {
         expect(filtered.some(section => !!section.items)).toBeFalsy();
       });

docs_app/src/app/custom-elements/contributor/contributor.component.ts

@@ -24,7 +24,7 @@ import { CONTENT_URL_PREFIX } from 'app/documents/document.service';
                     <a *ngIf="person.github" mat-button class="icon"
                         href="{{person.github}}" target="_blank" (click)="$event.stopPropagation()">
                         <span class="fa fa-github fa-2x" aria-hidden="true"></span>
-                        <span class="sr-only">Github {{person.name}}</span>
+                        <span class="sr-only">GitHub {{person.name}}</span>
                     </a>
                     <a *ngIf="person.website" mat-button class="icon"
                         href="{{person.website}}" target="_blank" (click)="$event.stopPropagation()">

docs_app/src/app/custom-elements/elements-loader.spec.ts

@@ -194,7 +194,7 @@ describe('ElementsLoader', () => {
       expect(definedSpy).not.toHaveBeenCalled();
     }));
 
-    it('should fail if defining the the custom element fails', fakeAsync(() => {
+    it('should fail if defining the custom element fails', fakeAsync(() => {
       let state = 'pending';
       elementsLoader.loadCustomElement('element-b-selector').catch(e => state = `rejected: ${e}`);
       flushMicrotasks();

docs_app/src/app/custom-elements/operator-decision-tree/operator-decision-tree.component.ts

@@ -21,14 +21,14 @@ import { OperatorDecisionTreeService } from './operator-decision-tree.service';
       </ng-container>
       <div>
         <ng-container *ngFor="let option of options$ | async">
-          <ng-container *ngIf="option.options; else operatorTempalte">
+          <ng-container *ngIf="option.options; else operatorTemplate">
             <button class="option mat-body-1" (click)="selectOption(option.id)" [@flyIn]>
               <mat-card matRipple>
                 {{ option.label }}
               </mat-card>
             </button>
           </ng-container>
-          <ng-template #operatorTempalte>
+          <ng-template #operatorTemplate>
             <p *ngIf="option.method" class="mat-body-1">
               You want the {{ option.method }} of the {{ option.docType }}
               <a href="{{ option.path }}#{{ option.method }}">{{ option.label }}</a
@@ -48,7 +48,7 @@ import { OperatorDecisionTreeService } from './operator-decision-tree.service';
         <p>Oops! There was an issue loading the decision tree.. we're real sorry about that. Please try reloading the page.</p>
         <p>
           You can also try
-          <a href="https://github.com/ReactiveX/rxjs/issues/new?template=documentation.md" target="_blank">submitting an issue on Github</a
+          <a href="https://github.com/ReactiveX/rxjs/issues/new?template=documentation.md" target="_blank">submitting an issue on GitHub</a
           >.
         </p>
       </div>

docs_app/src/app/custom-elements/operator-decision-tree/operator-decision-tree.service.spec.ts

@@ -31,7 +31,7 @@ describe('OperatorDecisionTreeService', () => {
   });
 
   describe('currentSentence$', () => {
-    const initialSetence = 'Start by choosing an option from the list below.';
+    const initialSentence = 'Start by choosing an option from the list below.';
 
     beforeEach(() => {
       dataServiceStub.getDecisionTree$.and.returnValue(
@@ -44,7 +44,7 @@ describe('OperatorDecisionTreeService', () => {
       it('should emit an initial sentence', () => {
         spyOn(service, 'selectOption');
         expect(service.currentSentence$).toBeObservable(
-          cold('x', { x: initialSetence })
+          cold('x', { x: initialSentence })
         );
         expect(service.selectOption).not.toHaveBeenCalled();
       });
@@ -83,7 +83,7 @@ describe('OperatorDecisionTreeService', () => {
           service.selectOption(treeNodeStubWithOptionsA.id);
           service.startOver();
           expect(service.currentSentence$).toBeObservable(
-            cold('x', { x: initialSetence })
+            cold('x', { x: initialSentence })
           );
         });
       });
@@ -191,7 +191,7 @@ describe('OperatorDecisionTreeService', () => {
 
   describe('hasError$', () => {
     describe('when the tree has no error', () => {
-      it('shoud not emit', () => {
+      it('should not emit', () => {
         dataServiceStub.getDecisionTree$.and.returnValue(
           cold('x', { x: treeStub })
         );

docs_app/src/app/documents/document.service.spec.ts

@@ -82,7 +82,7 @@ describe('DocumentService', () => {
       // Subsequent request for not-found document.
       logger.output.error = [];
       httpMock.expectOne(CONTENT_URL_PREFIX + 'file-not-found.json').flush(notFoundDoc);
-      expect(logger.output.error).toEqual([]); // does not report repeate errors
+      expect(logger.output.error).toEqual([]); // does not report repeated errors
       expect(currentDocument).toEqual(notFoundDoc);
     });
 

docs_app/src/app/navigation/navigation.service.spec.ts

@@ -65,7 +65,7 @@ describe('NavigationService', () => {
       expect(views2).toBe(views1);
       expect(views3).toBe(views1);
 
-      // Verfy that subsequent subscriptions did not trigger another request.
+      // Verify that subsequent subscriptions did not trigger another request.
       httpMock.expectNone({});
     });
 

docs_app/src/app/shared/scroll-spy.service.ts

@@ -37,7 +37,7 @@ export class ScrollSpiedElement implements ScrollItem {
 
   /*
    * @method
-   * Caclulate the `top` value, i.e. the value of the `scrollTop` property at which this element
+   * Calculate the `top` value, i.e. the value of the `scrollTop` property at which this element
    * becomes active. The current implementation assumes that window is the scroll-container.
    *
    * @param {number} scrollTop - How much is window currently scrolled (vertically).
@@ -72,7 +72,7 @@ export class ScrollSpiedElementGroup {
 
   /*
    * @method
-   * Caclulate the `top` value of each ScrollSpiedElement of this group (based on te current
+   * Calculate the `top` value of each ScrollSpiedElement of this group (based on te current
    * `scrollTop` and `topOffset` values), so that the active element can be later determined just by
    * comparing its `top` property with the then current `scrollTop`.
    *

docs_app/src/app/shared/select/select.component.spec.ts

@@ -73,7 +73,7 @@ describe('SelectComponent', () => {
     beforeEach(() => {
       host.options = options;
       host.showSymbol = true;
-      getButton().click(); // ensure the the options are visible
+      getButton().click(); // ensure the options are visible
       fixture.detectChanges();
     });
 

docs_app/src/app/shared/toc.service.spec.ts

@@ -79,7 +79,7 @@ describe('TocService', () => {
       expect(indices).toEqual([null, 42, 0, null, 7]);
     });
 
-    it('should reset athe active index (and unspy) when calling `reset()`', () => {
+    it('should reset the active index (and unspy) when calling `reset()`', () => {
       const indices: (number | null)[] = [];
 
       tocService.activeItemIndex.subscribe(i => indices.push(i));

docs_app/src/index.html

@@ -31,7 +31,7 @@
   <meta name="apple-mobile-web-app-status-bar-style" content="translucent">
 
   <script>
-    // Dynamically, pre-emptively, add `noindex`, which will be removed when the doc is ready and valid
+    // Dynamically, preemptively, add `noindex`, which will be removed when the doc is ready and valid
     var tag = document.createElement('meta'); tag.name = 'robots'; tag.content = 'noindex';
     document.head.appendChild(tag);
   </script>

docs_app/tools/decision-tree-generator/src/lib/decisionTreeReducer.spec.ts

@@ -20,7 +20,7 @@ describe('decisionTreeReducer', () => {
     });
 
     describe('that have options', () => {
-      it('should have an optons property that is an array of strings', () => {
+      it('should have an options property that is an array of strings', () => {
         for (const key in tree) {
           if (
             tree.hasOwnProperty(key) &&

docs_app/tools/decision-tree-generator/src/lib/extractInitialSequence.spec.ts

@@ -6,7 +6,7 @@ const tree = addUniqueId(mockRawTreeNodes);
 const initialSequence = extractInitialSequence(tree);
 
 describe('extractInitialSequence', () => {
-  describe('when givien a tree that has passed through addUniqueId', () => {
+  describe('when given a tree that has passed through addUniqueId', () => {
     it('will return an object that has an id of initial', () => {
       expect(initialSequence).toMatchObject({
         id: 'initial',

docs_app/tools/decision-tree-generator/src/lib/flattenApiList.ts

@@ -2,7 +2,7 @@ import { ApiListNode, FlattenedApiList } from './interfaces';
 import { isStable } from './helpers';
 
 /**
- * Flattens API List from the docs generation into a map with relavant properties.
+ * Flattens API List from the docs generation into a map with relevant properties.
  * Makes navigation easier.
  *
  * @export

docs_app/tools/decision-tree-generator/src/tree.yml

@@ -96,7 +96,7 @@
                 - label: where time windows are determined by a time duration
                   children:
                     - label: auditTime
-            - label: by emitting the last value as soon as enough silence has occured
+            - label: by emitting the last value as soon as enough silence has occurred
               children:
                 - label: where the silence duration threshold is determined by another Observable
                   children:

docs_app/tools/transforms/angular-api-package/index.js

@@ -57,7 +57,7 @@ module.exports = new Package('angular-api', [basePackage, typeScriptPackage])
   // Where do we get the source files?
   .config(function(readTypeScriptModules, readFilesProcessor, collectExamples, tsParser) {
 
-    // Tell TypeScript how to load modules that start with with `@angular`
+    // Tell TypeScript how to load modules that start with `@angular`
     tsParser.options.paths = { '@angular/*': [API_SOURCE_PATH + '/*'] };
     tsParser.options.baseUrl = '.';
 

docs_app/tools/transforms/angular-base-package/rendering/toId.spec.js

@@ -8,7 +8,7 @@ describe('toId filter', function() {
   it('should be called "toId"', function() { expect(filter.name).toEqual('toId'); });
 
   it('should convert a string to make it appropriate for use as an HTML id', function() {
-    expect(filter.process('This is a big string with €bad#characaters¢\nAnd even NewLines'))
-      .toEqual('This-is-a-big-string-with--bad-characaters--And-even-NewLines');
+    expect(filter.process('This is a big string with €bad#characters¢\nAnd even NewLines'))
+      .toEqual('This-is-a-big-string-with--bad-characters--And-even-NewLines');
   });
 });

docs_app/tools/transforms/examples-package/services/parseArgString.js

@@ -23,8 +23,8 @@ module.exports = function parseArgString() {
     //["] Close match if "
     var rx = /[^\s'"]+|[']([^']*?)[']|["]([^"]*?)["]/gi;
     var value = str;
-    var unnammedArgs = [];
-    var args = {_: unnammedArgs};
+    var unnamedArgs = [];
+    var args = {_: unnamedArgs};
     var match, key;
     do {
       // Each call to exec returns the next regex match as an array
@@ -47,7 +47,7 @@ module.exports = function parseArgString() {
               key = key.substr(1);
             }
           } else {
-            unnammedArgs.push(arg);
+            unnamedArgs.push(arg);
             key = null;
           }
         }

docs_app/tools/transforms/links-package/services/getDocFromAlias.js

@@ -15,7 +15,7 @@ module.exports = function getDocFromAlias(aliasMap) {
   function getDocFromAlias(alias, originatingDoc) {
     return getDocFromAlias.disambiguators.reduce(
       // Run the disambiguators while there is more than 1 doc found
-      (docs, disambiguater) => docs.length > 1 ? disambiguater(alias, originatingDoc, docs) : docs,
+      (docs, disambiguator) => docs.length > 1 ? disambiguator(alias, originatingDoc, docs) : docs,
       // Start with the docs that match the alias
       aliasMap.getDocs(alias)
     );

docs_app/tools/transforms/target-package/services/targetEnvironments.spec.js

@@ -49,7 +49,7 @@ describe('target inline-tag-def', function() {
   });
 
   describe('deactivate', function() {
-    it('should deactive an already allowed target', function() {
+    it('should deactivate an already allowed target', function() {
       te.addAllowed('a', true);
       te.addAllowed('b', false);
       te.addAllowed('c');

integration/import/fixtures/commonjs/index.js

@@ -6,11 +6,11 @@ const operators = require('rxjs/operators');
 const rxSnapshot = require('./rx.json');
 const operatorsSnapshot = require('./operators.json');
 
-const coldObservalbe = require('rxjs/internal/testing/ColdObservable');
+const coldObservable = require('rxjs/internal/testing/ColdObservable');
 
 assert.ok(rx, 'main export should exists');
 assert.ok(operators, 'operator export should exists');
-assert.ok(coldObservalbe, 'internal can be imported');
+assert.ok(coldObservable, 'internal can be imported');
 
 assert.deepStrictEqual(Object.keys(rx).sort(), rxSnapshot.sort(), 'main export should include exports');
 assert.deepStrictEqual(Object.keys(operators).sort(), operatorsSnapshot.sort(), 'operator export should include exports');

spec-dtslint/operators/concatMap-spec.ts

@@ -18,11 +18,11 @@ it('should infer correctly by using the resultSelector second parameter', () =>
 });
 
 it('should support a resultSelector that takes an inner index', () => {
-  const o = of(1, 2, 3).pipe(concatMap(p => of(Boolean(p)), (a, b, innnerIndex) => a)); // $ExpectType Observable<number>
+  const o = of(1, 2, 3).pipe(concatMap(p => of(Boolean(p)), (a, b, innerIndex) => a)); // $ExpectType Observable<number>
 });
 
 it('should support a resultSelector that takes an inner and outer index', () => {
-  const o = of(1, 2, 3).pipe(concatMap(p => of(Boolean(p)), (a, b, innnerIndex, outerIndex) => a)); // $ExpectType Observable<number>
+  const o = of(1, 2, 3).pipe(concatMap(p => of(Boolean(p)), (a, b, innerIndex, outerIndex) => a)); // $ExpectType Observable<number>
 });
 
 it('should support an undefined resultSelector', () => {

spec-dtslint/operators/concatMapTo-spec.ts

@@ -26,11 +26,11 @@ it('should infer correctly by using the resultSelector second parameter', () =>
 });
 
 it('should support a resultSelector that takes an inner index', () => {
-  const o = of(1, 2, 3).pipe(concatMapTo(of('foo'), (a, b, innnerIndex) => a)); // $ExpectType Observable<number>
+  const o = of(1, 2, 3).pipe(concatMapTo(of('foo'), (a, b, innerIndex) => a)); // $ExpectType Observable<number>
 });
 
 it('should support a resultSelector that takes an inner and outer index', () => {
-  const o = of(1, 2, 3).pipe(concatMapTo(of('foo'), (a, b, innnerIndex, outerIndex) => a)); // $ExpectType Observable<number>
+  const o = of(1, 2, 3).pipe(concatMapTo(of('foo'), (a, b, innerIndex, outerIndex) => a)); // $ExpectType Observable<number>
 });
 
 it('should support an undefined resultSelector', () => {