Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: grammar and spelling fixes #1086

Merged
merged 2 commits into from
Apr 20, 2022
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 2 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -211,8 +211,8 @@ adjust your eslint config as outlined

Note that unlike the type-checking rules in `@typescript-eslint/eslint-plugin`,
the rules here will fallback to doing nothing if type information is not
available, meaning its safe to include them in shared configs that could be used
on JavaScript and TypeScript projects.
available, meaning it's safe to include them in shared configs that could be
used on JavaScript and TypeScript projects.

Also note that `unbound-method` depends on `@typescript-eslint/eslint-plugin`,
as it extends the original `unbound-method` rule from that plugin.
Expand Down
2 changes: 1 addition & 1 deletion docs/rules/no-conditional-expect.md
Original file line number Diff line number Diff line change
Expand Up @@ -107,7 +107,7 @@ While you can use `expect.assertions` & `expect.hasAssertions` for these
situations, they only work with `expect`.

A better way to handle this situation is to introduce a wrapper to handle the
catching, and otherwise returns a specific "no error thrown" error if nothing is
catching, and otherwise return a specific "no error thrown" error if nothing is
thrown by the wrapped function:

```typescript
Expand Down
3 changes: 1 addition & 2 deletions docs/rules/no-deprecated-functions.md
Original file line number Diff line number Diff line change
Expand Up @@ -37,8 +37,7 @@ Originally, the `requireActual` & `requireMock` the `requireActual`&

These functions were later moved onto the `jest` object in order to be easier
for type checkers to handle, and their use via `require` deprecated. Finally,
the release of Jest 26 saw them removed from the `require` function all
together.
the release of Jest 26 saw them removed from the `require` function altogether.

### `jest.runTimersToTime`

Expand Down
2 changes: 1 addition & 1 deletion docs/rules/no-identical-title.md
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ fix.

## Rule Details

This rule looks at the title of every test and test suites. It will report when
This rule looks at the title of every test and test suite. It will report when
two test suites or two test cases at the same level of a test suite have the
same title.

Expand Down
4 changes: 2 additions & 2 deletions docs/rules/no-jasmine-globals.md
Original file line number Diff line number Diff line change
Expand Up @@ -9,8 +9,8 @@ API.

### Rule details

This rule reports on any usage of Jasmine globals which is not ported to Jest,
and suggests alternative from Jest's own API.
This rule reports on any usage of Jasmine globals, which is not ported to Jest,
and suggests alternatives from Jest's own API.

### Default configuration

Expand Down
2 changes: 1 addition & 1 deletion docs/rules/no-jest-import.md
Original file line number Diff line number Diff line change
Expand Up @@ -17,4 +17,4 @@ first place.

## Further Reading

\*[The Jest Object](https://facebook.github.io/jest/docs/en/jest-object.html)
- [The Jest Object](https://facebook.github.io/jest/docs/en/jest-object.html)
4 changes: 2 additions & 2 deletions docs/rules/no-large-snapshots.md
Original file line number Diff line number Diff line change
Expand Up @@ -118,8 +118,8 @@ External). Use `inlineMaxSize` for
[Inline Snapshots](https://jestjs.io/docs/en/snapshot-testing#inline-snapshots)
size and `maxSize` for
[External Snapshots](https://jestjs.io/docs/en/snapshot-testing#snapshot-testing-with-jest).
If only `maxSize` is provided on options, the value of `maxSize` will be used to
both snapshot types (Inline and External).
If only `maxSize` is provided on options, the value of `maxSize` will be used
for both snapshot types (Inline and External).

Since `eslint-disable` comments are not preserved by Jest when updating
snapshots, you can use the `allowedSnapshots` option to have specific snapshots
Expand Down
2 changes: 1 addition & 1 deletion docs/rules/no-standalone-expect.md
Original file line number Diff line number Diff line change
Expand Up @@ -61,7 +61,7 @@ describe('a test', () => {
```

\*Note that this rule will not trigger if the helper function is never used even
thought the `expect` will not execute. Rely on a rule like no-unused-vars for
though the `expect` will not execute. Rely on a rule like no-unused-vars for
this case.

### Options
Expand Down
2 changes: 1 addition & 1 deletion docs/rules/prefer-comparison-matcher.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Suggest using the built-in comparison matchers (`prefer-comparison-matcher`)

Jest has a number of built-in matchers for comparing numbers which allow for
Jest has a number of built-in matchers for comparing numbers, which allow for
more readable tests and error messages if an expectation fails.

## Rule details
Expand Down
2 changes: 1 addition & 1 deletion docs/rules/prefer-equality-matcher.md
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
# Suggest using the built-in equality matchers (`prefer-equality-matcher`)

Jest has built-in matchers for expecting equality which allow for more readable
Jest has built-in matchers for expecting equality, which allow for more readable
tests and error messages if an expectation fails.

## Rule details
Expand Down
2 changes: 1 addition & 1 deletion docs/rules/prefer-hooks-on-top.md
Original file line number Diff line number Diff line change
@@ -1,7 +1,7 @@
# Suggest having hooks before any test cases (`prefer-hooks-on-top`)

While hooks can be setup anywhere in a test file, they are always called in a
SimenB marked this conversation as resolved.
Show resolved Hide resolved
specific order which means it can be confusing if they're intermixed with test
specific order, which means it can be confusing if they're intermixed with test
cases.

This rule helps to ensure that hooks are always defined before test cases.
Expand Down
4 changes: 2 additions & 2 deletions docs/rules/prefer-lowercase-title.md
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,8 @@ it('Uppercase description');

### `allowedPrefixes`

This array option allows specifying prefixes which contain capitals that titles
can start with. This can be useful when writing tests for api endpoints, where
This array option allows specifying prefixes, which contain capitals that titles
can start with. This can be useful when writing tests for API endpoints, where
you'd like to prefix with the HTTP method.

By default, nothing is allowed (the equivalent of `{ "allowedPrefixes": [] }`).
Expand Down