Tslint: नो-अप्रयुक्त-चर टाइप एरर: नल की संपत्ति '1' नहीं पढ़ सकता है

को निर्मित 17 मई 2018 · 14टिप्पणियाँ · स्रोत: palantir/tslint

बग रिपोर्ट

__TSLint संस्करण__: 5.10.0
__टाइपस्क्रिप्ट संस्करण__: 2.8.1
__TSLint के माध्यम से चल रहा है__: CLI

टाइपस्क्रिप्ट कोड पंक्तिबद्ध किया जा रहा है

import { A, B, C } from './file';

console.log('No imports used');

export const D = 4;

tslint.json कॉन्फ़िगरेशन के साथ:

{
  "rules": {
    "no-unused-variable": [true, {"ignore-pattern": "^_"}]
  }
}

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

The 'no-unused-variable' rule threw an error in '/Users/andrew.mitchell/Documents/Projects/test/test.ts':
TypeError: Cannot read property '1' of null
    at walk (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/rules/noUnusedVariableRule.js:105:54)
    at Rule.AbstractRule.applyWithFunction (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/language/rule/abstractRule.js:39:9)
    at Rule.applyWithProgram (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/rules/noUnusedVariableRule.js:32:21)
    at Linter.applyRule (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/linter.js:194:29)
    at /Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/linter.js:139:85
    at Object.flatMap (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/utils.js:151:29)
    at Linter.getAllFailures (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/linter.js:139:32)
    at Linter.lint (/Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/linter.js:99:33)
    at /Users/andrew.mitchell/Documents/test/node_modules/tslint/lib/runner.js:209:32
    at step (/Users/andrew.mitchell/Documents/test/node_modules/tslint/node_modules/tslib/tslib.js:133:27)

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

नियम को लाइन 1 के लिए All imports in import declaration are unused. चेतावनी देनी चाहिए।

यह noUnusedVariableRule.ts#L123 के कारण होता है क्योंकि failure संदेश All imports in import declaration are unused. , जिसमें रेगेक्स को खोजने के लिए इसमें एक चर नाम नहीं है।

यह नियम सही ढंग से काम करता है यदि ignore-pattern विकल्प निर्दिष्ट नहीं है क्योंकि यह चर नाम की जांच करने का प्रयास नहीं करेगा।

Fixed Bug

स्रोत

hotforfeature

👍5 👎1

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

noUnusedLocals और noUnusedParameters को compilerOptions में परिभाषित करना tslint के no-unused-variable । अभी तक, अप्रयुक्त संस्करण या तो बिल्ड को क्रैश कर देते हैं या किसी का ध्यान नहीं जाता है और अब कोई चेतावनी मोड नहीं है

kachkaev 27 जुल॰ 2018

👍21

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

अच्छा खोज @hotforfeature। मैंने इसे https://github.com/palantir/tslint/pull/3919 में संबोधित करना शुरू कर दिया क्योंकि यह TS 2.9 में आने वाले कुछ बदलावों से संबंधित है जो केवल इस मुद्दे को बढ़ा देगा।

मेरा पीआर एक अपवाद को फेंके जाने से रोकता है, हालांकि, यह आपके द्वारा बताए गए मामलों में केवल ignorePattern को अनदेखा करता है जो कि बहुत अच्छा नहीं है। मेरा मानना है कि हमें कुछ जटिल कोड जोड़ने की आवश्यकता होगी जैसे कि हम अभी आयात ऑटोफिक्स के लिए क्या करते हैं: /

jkillian 18 मई 2018

मैं सोच रहा हूं कि क्या हमें इस नियम को हटाने की बात करनी चाहिए। tsc अब इसकी चेतावनियों को अनदेखा करने का एक तरीका प्रदान करता है, और यह एक IDE के माध्यम से इसकी चेतावनियों के लिए स्वचालित सुधारों के लिए समर्थन प्रदान करता है।

जहां तक मैं देख सकता हूं, इस नियम का मुख्य लाभ ignore-pattern , लेकिन इसे सही ढंग से लागू करना मुश्किल है जिस तरह से अब tsc डायग्नोस्टिक्स पर निर्भर रहने के लिए नियम लिखा गया है। TSLint कुछ फ़ाइलों में नियमों को अक्षम करने के संबंध में tsc w/से थोड़ा अधिक लचीला है। @suchanlee इस सब पर कोई विचार? मुझे पता है कि आपने हाल ही में यह नियम तय किया है

jkillian 18 मई 2018

इस नियम के लिए मेरा प्राथमिक उपयोग केस लाइनिंग करते समय गिट हुक में प्रतिबद्ध होने पर समस्याओं को स्वत: ठीक करना है। तो मुझे अभी भी लगता है कि यहां बहुत अच्छा मूल्य है, भले ही टीएससी आईडीई एकीकरण के माध्यम से कुछ प्रदान करता हो।

hotforfeature 18 मई 2018

👍1

@JKillian मुझे लगता है कि टाइपस्क्रिप्ट के कई संस्करणों और टाइपस्क्रिप्ट व्यवहार में निरंतर परिवर्तन के लिए TSLint के समर्थन के समर्थन के लिए नियम तेजी से महंगा होने लगा है। और चूंकि टाइपस्क्रिप्ट सुविधा का समर्थन करता है, इसलिए हमें इसका उपयोग करने की ओर झुकना चाहिए। लेकिन चूंकि TS के पुराने संस्करण इसका समर्थन नहीं करते हैं, इसलिए मुझे नहीं लगता कि हमें इसे हटा देना चाहिए। हालांकि मुझे लगता है कि हमें नियम को बहिष्कृत करने के बारे में सोचना शुरू करना चाहिए और टीएस के साथ मौजूदा टीएसलिंट वर्कफ़्लो का बेहतर समर्थन करने के लिए टीएस लोगों के साथ काम करना चाहिए।

suchanlee 19 मई 2018

👎2 👍2

मैं @suchanlee के प्रस्ताव को दूसरा

aervin 24 मई 2018

👍2

@hotforfeature - अपने समझें । हालांकि, नियम की जटिलता के कारण, मुझे लगता है कि हम इसे पदावनत करने जा रहे हैं (देखें #3919)।

हमेशा की तरह, नियम के स्रोत कोड की प्रतिलिपि बनाने और इसे बाहरी पैकेज में स्थानांतरित करने के लिए किसी का भी स्वागत है और इस तरह इसका उपयोग/सुधार करते रहें!

jkillian 31 मई 2018

👍2

हांलांकि इसकी कीमत के बारे निश्चित नहीं हूँ:

{
  "compilerOptions":  {
    "noUnusedLocals": true,
    "noUnusedParameters": true,
  }
}

tsconfig में उपरोक्त का उपयोग करने से no-unused-variable लिंट नियम को अक्षम किया जा सकता है।

killtheliterate 31 मई 2018

👎9 🎉1

no-unused-variable अब पदावनत कर दिया गया है, और उपरोक्त compilerOptions अब आधिकारिक समाधान है।

giladgray 14 जून 2018

👎14

kachkaev 27 जुल॰ 2018

👍21

@giladgray, @killtheliterate लिखा है:

tsconfig में उपरोक्त का उपयोग करने से अप्रयुक्त-चर लिंट नियम को अक्षम किया जा सकता है।

मैं इस नियम को अक्षम नहीं करना चाहता, क्योंकि जैसा कि @kachkaev ने लिखा है

noUnusedLocals और noUnusedParameters को compilerOptions में परिभाषित करना tslint के no-unused-variable

मेरा उपयोग-मामला जेनरेट कोड है। मैं चाहता हूं कि मेरा मैन्युअल रूप से लिखित कोड इस नियम का पालन करे, लेकिन कोड-जनरेटर को अत्यधिक जटिल बनाने से बचने के लिए, मैं इस नियम को जेनरेट कोड में अक्षम करना चाहता हूं (या कम से कम जेनरेट कोड के कुछ अनुभागों में) - वह smth है जो कर सकता है ' टाइपस्क्रिप्ट कंपाइलर विकल्पों के साथ नहीं किया जाना चाहिए

atsu85 31 जुल॰ 2018

👍1

कम से कम, इस बहिष्करण को TSLint वेबसाइट पर प्रलेखित किया जाना चाहिए, लेकिन मैं दृढ़ता से सहमत हूं कि यह बहिष्करण समय से पहले हुआ था। फिर से विचार करने की कोई योजना? क्या सामुदायिक योगदान स्वीकार किया जाएगा?

aleclarson 14 जन॰ 2019

👍1

यह बहिष्कार हास्यास्पद है। संकलक झंडे एक भयानक प्रतिस्थापन हैं। वे निर्माण को तोड़ते हैं और खुद को ठीक नहीं करते हैं। लोगों को इंगित करने के लिए यह एक उचित समाधान कैसे है? यदि ignore-pattern विकल्प अभी काम करना बहुत कठिन है, तो इसके बजाय उसे हटा दें।

maxbogue 19 जन॰ 2019

दोस्तों - बस इसे https://github.com/palantir/tslint/issues/4232 से लिंक कर रहे हैं no-unused-variable लिए पर्याप्त प्रतिस्थापन हैं। नियम का मूल कार्यान्वयन अन्य TS टूलिंग के साथ अच्छा नहीं चला और इसे अक्षम करना पड़ा। #4232 ट्रैक इसे फिर से सक्षम करने का तरीका ढूंढ रहे हैं।

इस बीच, आप अंतर्निहित चेक का उपयोग करने के लिए tsc --noEmit --noUnusedLocals --noUnusedParameters जैसे कुछ का उपयोग एक अलग टूल के रूप में कर सकते हैं। हां, यह विन्यास योग्य no-unused-variables जितना अच्छा नहीं है।

JoshuaKGoldberg 19 जन॰ 2019

❤3

#4100 और #4232 . में चर्चा जारी रखें

adidahiya 12 फ़र॰ 2019

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

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