Skip to content
This repository was archived by the owner on Nov 16, 2023. It is now read-only.
This repository was archived by the owner on Nov 16, 2023. It is now read-only.

Disable rule is often not shown #22

Closed
@arogg

Description

@arogg

I believe disabling a rule on a line by line basis should always be an option. Currently it is not. Most of the time the option to disable a rule is not available ...even when the quick fix option to disable all is available.

I was about to migrate from the old project at https://github.com/angelozerr/tslint-language-service and I am wondering if this PR can be included into this project: angelozerr/tslint-language-service#87 ?

I'm back to the old project for now where I have manually added this PR to my personal installation. But I just wanted to let you know because I just tried out this version and this is what was missing for me.

Or maybe I am missing something? Without the option to disable a rule, I must always look up the corresponding rule and type the comment to disable it by hand. Surely this is not intended? :)

Thank you.

Edit. example

const somevar = null as any;
const somevar2 = require(somevar);

This code shows 2 tslint errors (no-any and no-var-requires), but non of the disable rule quick fixes will show!

Metadata

Metadata

Assignees

No one assigned

    Labels

    help wantedExtra attention is needed

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions