Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[ban-ts-comment] require comment descriptions by default #2146

Closed
Labels
breaking changeThis change will require a new major version to be releaseddefault rule optionsDiscussions about default rule optionspackage: eslint-pluginIssues related to @typescript-eslint/eslint-plugin
Milestone
@dimitropoulos

Description

@dimitropoulos

now that#2099 is merged and users can specifyallow-with-description for the comment directives specified inban-ts-comment, this issue serves as areminder to make this change for 4.0.

with the next major, we can consider setting the default tots-expect-error: 'allow-with-comment'
@bradzacher2020 05 25

Aside from the tests to be updated, the breaking change will look something like this:

const defaultOptions: Options = {- 'ts-expect-error': true,+ 'ts-expect-error': 'allow-with-description',  'ts-ignore': true,  'ts-nocheck': true,- 'ts-check': false,+ 'ts-check': 'allow-with-description',  minimumDescriptionLength: 3,};

Metadata

Metadata

Assignees

No one assigned

    Labels

    breaking changeThis change will require a new major version to be releaseddefault rule optionsDiscussions about default rule optionspackage: eslint-pluginIssues related to @typescript-eslint/eslint-plugin

    Type

    No type

    Projects

    No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions


      [8]ページ先頭

      ©2009-2025 Movatter.jp