Tslint: समर्थित `नो-अप्रयुक्त-चर` नियम के लिए प्रदर्शित चेतावनी

को निर्मित 13 सित॰ 2017 · 3टिप्पणियाँ · स्रोत: palantir/tslint

बग रिपोर्ट

__टीएसलिंट संस्करण__:

$ $(yarn bin)/tslint --version
5.7.0

__टाइपस्क्रिप्ट संस्करण__:

$ $(yarn bin)/tsc --version
Version 2.5.2

__TSLint के माध्यम से चल रहा है__: CLI

वास्तविक व्यवहार

मैं मौजूदा कोड आधार पर tslint चला रहा हूं।

मेरे प्रोजेक्ट में no-unused-variable नियम सेट है (प्रीसेट-जैसे पैकेज के उपयोग के माध्यम से)।

मेरे प्रोजेक्ट में टाइपस्क्रिप्ट के लिए सक्षम noUnusedLocals और noUnusedParameters कंपाइलर विकल्प भी हैं (इसके खिलाफ, प्रीसेट-जैसे पैकेज के उपयोग के माध्यम से)।

मैं कभी-कभी अपने नियमों को अपडेट करता हूं और अपने कोड को स्वतः ठीक करने के लिए tslint का --fix विकल्प चलाता हूं।

इसमें no-unused-variable नियम की ऑटो-फिक्स सुविधा से लाभ शामिल है।

मेरे कोड बेस पर tslint --fix चलाते समय उदाहरण आउटपुट:

$ $(yarn bin)/tslint --project tsconfig.json --config tslint.json --fix src/**/*.ts
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.

अपेक्षित् व्यवहार

जैसा कि यहां बताया गया no-unused-variable नियम को हटा दिया गया था क्योंकि उपयोगकर्ता (स्वयं सहित) ऑटो-फिक्स सुविधा से लाभान्वित होते हैं।

हालांकि चेतावनी का तात्पर्य है कि मुझे नियम का बिल्कुल भी उपयोग नहीं करना चाहिए, और कंसोल के लिए बहुत सारे अनावश्यक आउटपुट का कारण बनता है, जो अन्य महत्वपूर्ण जानकारी जैसे कोड में गैर-ठीक करने योग्य समस्याओं को अस्पष्ट कर सकता है।

मेरा मानना है कि चेतावनी को हटाना ठीक रहेगा।

P3 Accepting PRs Bug

स्रोत

ghost

सबसे उपयोगी टिप्पणी

@destroyerofbuilds जो एक अच्छे बिंदु की तरह लगता है - आप मुख्य रूप से ऑटो-फ़िक्स कार्यक्षमता के लिए no-unused-variable का उपयोग कर रहे हैं, इसलिए चेतावनी अप्रासंगिक है। मुझे लगता है कि हमें चेतावनी (पीआर को स्वीकार करते हुए) को हटा देना चाहिए। इस उपयोग पैटर्न का वर्णन करने के लिए नियम दस्तावेज़ों को अपडेट करना भी अच्छा होगा।

adidahiya 14 सित॰ 2017

👍2

सभी 3 टिप्पणियाँ

मेरे लिए, अप्रयुक्त-चर कुछ भी नहीं करता है ;(
संस्करण 5.7.0

यह मेरा विन्यास है

{
    "extends": [
        "tslint:latest",
        "tslint-react",
        "tslint-config-prettier"
    ],
    "rules": {
        "ban-types": false,
        "interface-name": null,
        "interface-over-type-literal": false,
        "jsx-boolean-value": null,
        "jsx-no-lambda": null,
        "max-classes-per-file": [],
        "member-access": false,
        "no-namespace": null,
        "no-unused-expression": [false],
        "no-unused-variable": true,
        "no-var-requires": false,
        "object-literal-shorthand": false,
        "object-literal-sort-keys": false,
        "only-arrow-functions": null,
        "ordered-imports": null,
        "prefer-const": false,
        "variable-name": false,
        "no-invalid-template-strings": false,
        "no-submodule-imports": false
    }
}

tomitrescak 14 सित॰ 2017

adidahiya 14 सित॰ 2017

👍2

@tomitrescak आप

adidahiya 14 सित॰ 2017

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

Tslint: समर्थित `नो-अप्रयुक्त-चर` नियम के लिए प्रदर्शित चेतावनी

बग रिपोर्ट

वास्तविक व्यवहार

अपेक्षित् व्यवहार

सबसे उपयोगी टिप्पणी

सभी 3 टिप्पणियाँ

संबंधित मुद्दों