Skip to content

Prefer Jest's toBe true/false matcher.#1639

Merged
fb55 merged 1 commit intocheeriojs:mainfrom
XhmikosR:jest
Jan 4, 2021
Merged

Prefer Jest's toBe true/false matcher.#1639
fb55 merged 1 commit intocheeriojs:mainfrom
XhmikosR:jest

Conversation

@XhmikosR
Copy link
Copy Markdown
Contributor

@XhmikosR XhmikosR commented Jan 3, 2021

Also, remove some trailing spaces from descriptions.

Might be worth adding https://github.com/jest-community/eslint-plugin-jest later :)

Also, remove some trailing spaces from descriptions.
@5saviahv
Copy link
Copy Markdown
Contributor

5saviahv commented Jan 3, 2021

Interesting toBe(true) vs toBeTruthy() vs toBeTrue()

@fb55 fb55 merged commit 4859684 into cheeriojs:main Jan 4, 2021
@fb55
Copy link
Copy Markdown
Member

fb55 commented Jan 4, 2021

Nice one 🙂

Might be worth adding https://github.com/jest-community/eslint-plugin-jest later :)

Good call, looking into it!

@XhmikosR XhmikosR deleted the jest branch January 4, 2021 05:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants