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.

Sign up for GitHub

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

Jump to bottom

Use fewer type assertions #516

Open

danielnixon opened this issue Jan 31, 2023 · 0 comments

Collaborator

danielnixon commented Jan 31, 2023

For example,

  // eslint-disable-next-line @typescript-eslint/consistent-type-assertions
  const d = new Date(value as number | string | Date);

can probably become:

const d = new Date(typeof value === "string" ? value : +value);

(courtesy of:)

    /**
     * Converts a Date object to a number.
     */
    [Symbol.toPrimitive](hint: "number"): number;

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment