Require explanation on eslint-disable and ts-expect-error

[fregante]

Not allowed

// eslint-disable-next-line no-var
var iLikeVar = true;

// @ts-expect-error
const iLikeErrors = undefined.undefined;

Allowed

// eslint-disable-next-line no-var -- TODO: I'll look into this later
var iLikeVar = true;

// @ts-expect-error -- TypeScript is actually wrong here, this totes works
const iLikeErrors = undefined.undefined;

[fregante]

The second one is already exists 🎉

https://github.com/typescript-eslint/typescript-eslint/blob/master/packages/eslint-plugin/docs/rules/ban-ts-comment.md

But I don't see it here? 🤔

https://github.com/xojs/eslint-config-xo-typescript/blob/main/index.js

[sindresorhus]

Is there a rule to enforce it in ESLint (for the non-TS case)?

[fregante]

Require explanation on eslint-disable

Still not enabled because of eslint-plugin-eslint-comments’ inactivity. Hopefully a new rule will come out. Unicorn?