eslint-plugin-unicorn

eslint-plugin-unicorn copied to clipboard

Rule proposal: `no-template-in-string`

1

### Description Sometimes when I'm working with error messages, I'll have something like `Failed to add user to _something_`. In some cases it will be more detailed, but not from...

DaniFoldi

Mention Node version support in `prefer-node-protocol` rule

1

Document the supported Node versions for the node protocol import feature to reduce potential confusion or mistaken adoption of this lint rule. While eslint-plugin-unicorn only supports Node 14.18 and above:...

bmish

When using Remix framework, rule `text-encoding-identifier-case` reports error: ```ts import type {MetaFunction} from '@remix-run/node'; /* ... */ export const meta: MetaFunction = () => ({ charset: 'utf-8', //

jakubmazanec

Rule proposal: Prepend type variables in unions: `A | {B: string}` instead of `{B: string} | A`

5

### Description Large inline type definitions may shadow trailing unions with pre-existing types. I think that there was a similar rule for regular javascript objects (I thought it was [object-shorthand](https://eslint.org/docs/rules/object-shorthand))...

fregante

Fixes #1868 I realized that having it as a setting is not a right thing, because linter fixes should be essentially the same code, just restyled. Trimming trailing spaces essentially...

mnaoumov

`func-style` like rule

11

See: http://eslint.org/docs/rules/func-style My personal preference is to prefer `declaration` style, as it allows hoisting (and just looks better to me). The only issue is when you want to reassign: ```...

jamestalmage

prevent-abbreviations: TypeScript type references are not considered by autofixer, interfaces and type aliases not checked

1

```ts enum Btn { Hello = 123, } const test1 = Btn.Hello let test2: Btn.Hello ``` Gets autofixed to: ```ts enum Button { Hello = 123, } const test1 =...

felixfbecker

ESLint with plugin unicorn brakes in a NX monorepo when having packages on a deeper folder level. It works fine in packages like `/libs/first-level` (path `@nx-monorepo/first-level`) but throws this error...

dev-justtrade

`prefer-array-some` should also check for misuse of `Array.findIndex()`

2

### Description **prefer-array-some** checks for situations in which `Array.filter(...).length > 0` and `Array.find(...) != null` could be better written as `Array.some(...)` It would be useful to also check for a...

KurtPreston

https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1815#issuecomment-1126894101 https://github.com/sindresorhus/eslint-plugin-unicorn/issues/1868

JounQin