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

chore: fix invalid use of typeof operator #3076

Merged
merged 1 commit into from
Dec 13, 2021
Merged

Conversation

nbbeeken
Copy link
Contributor

Description

What is changing?

Adds an eslint rule to catch my mistake. typeof x must be checked against one of the known string results.

What is the motivation for this change?

A superficial bug where we might have been defining properties and setting them to undefined when we intented to filter them out.

Since this bug has no virtually no impact, I decided against a ticket and even entering it into the changelog, happy to adjust.

Double check the following

  • Ran npm run check:lint script
  • Self-review completed using the steps outlined here
  • PR title follows the correct format: <type>(NODE-xxxx)<!>: <description>
  • Changes are covered by tests
  • New TODOs have a related JIRA ticket

@nbbeeken nbbeeken added the Team Review Needs review from team label Dec 13, 2021
@nbbeeken nbbeeken merged commit 9d84765 into main Dec 13, 2021
@nbbeeken nbbeeken deleted the fix/invalid-typeof branch December 13, 2021 19:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team Review Needs review from team
Projects
None yet
3 participants