Jshint: टैब इंडेंटेशन कई नियम तोड़ता है

रिपोर्ट के लिए धन्यवाद! समस्या lex.js की लाइन 1610 से उपजी है:

this.input = this.input.replace(/\t/g, state.tab);

यह बहुत मौलिक लगता है कि कितने स्टाइल-संबंधित लाइनिंग नियम लागू किए गए हैं। वे कुछ समय के लिए बहिष्कृत कर दिए गए हैं, लेकिन हम अभी भी एक नए प्रमुख संस्करण पर आगे बढ़ने के लिए तैयार नहीं हैं। तो इस बीच, हमें एक ऐसा समाधान तैयार करना होगा जो उस व्यवहार को बरकरार रखे।

मेरा प्रारंभिक विचार इस "प्रभावी ऑफसेट" को column नामक टोकन ऑब्जेक्ट की एक नई संपत्ति के रूप में ट्रैक करना है। इसका उपयोग उन शैली-संबंधी चेतावनियों के लिए किया जा सकता है। फिर, "सत्य" वर्ण ऑफ़सेट का वर्णन करने के लिए character विशेषता को फिर से लागू किया जा सकता है, जहां प्रत्येक कोड बिंदु (टैब या अन्यथा) गिनती को ठीक 1 से बढ़ाता है। इस मान का उपयोग चेतावनियां जारी करने के लिए किया जाएगा। रिपोर्टर प्लगइन्स तब नियंत्रण में होंगे कि उन्होंने टैब चरित्र को कैसे प्रस्तुत किया, और वे तदनुसार रिपोर्ट किए गए "कॉलम" संख्या की व्याख्या कर सकते थे।

क्या यह आपको अच्छा लगता है, @Arcanemagus?

यह मेरे लिए सही लगता है, कम से कम सतह पर जेएसहिंट के आंतरिक के वास्तविक ज्ञान के साथ नहीं। आदर्श रूप से मुझे केवल सही कॉलम गिनती प्राप्त करने की कुछ विधि चाहिए (जहां प्रत्येक कोड बिंदु एक कॉलम के रूप में गिना जाता है)।

अर्ध-सामान्य त्रुटि के साथ कुछ पुराने पार्सर बग के समाधान के कारण इस चेतावनी को अनिवार्य रूप से अनदेखा किए जाने के कारण यह कुछ समय के लिए छिपा हुआ था। जब मैंने उपयोगकर्ताओं को चेतावनी को बहुत अच्छे तरीके से फिर से लागू किया तो मैंने गलती से उन्हें _hidden_ ​​कर दिया था, और हाल ही में इसे फिर से दृश्यमान बनाने के लिए इसे ठीक किया।

ऐसा लगता है कि W030 , W033 , और W009 भी प्रभावित हैं। आपके विवरण से मुझे लगता है कि _all_ नियम प्रभावित हैं?

मैं भी इस समस्या का सामना कर रहा हूं और त्रुटियों और चेतावनियों दोनों पर दोहराने में सक्षम हूं। नियम/त्रुटि प्रकार कोई मायने नहीं रखता।

यहाँ jshint का एक उदाहरण है जो मुझे चेतावनी देने का प्रयास कर रहा है कि मेरा '==' होना चाहिए '==='। नारंगी रेखांकन के गलत स्थान पर ध्यान दें:

उम्मीद है कि यह इस मुद्दे पर आपके विचारों की पुष्टि करता है।

मुझे आज कुछ खुदाई करने के लिए कुछ समय मिला है, और मेरा मानना ​​है कि indent विकल्प को 1 पर सेट करने से वांछित परिणाम प्राप्त होंगे।

@Arcanemagus क्या आप अपने उपभोक्ताओं के लिए indent विकल्प के मूल्य से आगे निकलने में सक्षम हैं? यदि यह संभव है, तो मैं ऐसा करना चाहूंगा, क्योंकि मुझे यकीन नहीं है कि JSHint में डिफ़ॉल्ट मान को बदलने से इसके उपभोक्ता कैसे प्रभावित हो सकते हैं।

हम्म, वर्तमान में यह प्रोजेक्ट सीएलआई इंटरफ़ेस का उपयोग करता है जो ऐसा दिखता है। Node.js API का उपयोग करने के लिए इसे फिर से लिखना संभव है जो इसे संभव बना देगा।

• क्या indent मूल्य को बाध्य करने से उपभोक्ताओं द्वारा देखे जाने वाले परिणाम बदल जाएंगे? उस पैकेज का लक्ष्य जितना संभव हो उतना करीब होना है जितना वह स्वयं jshint चलाने का हो सकता है, लेकिन परिणाम संपादक में एकीकृत करना।
• क्या JSHint API में किसी फ़ाइल के कॉन्फ़िगरेशन को हथियाने के लिए एक छिपी हुई विधि है, या इसे लागू करने की आवश्यकता होगी? वर्तमान दस्तावेज ऐसा कोई कार्य नहीं दिखाता है।

कमांड लाइन तर्क जोड़ने के बारे में जो कॉन्फ़िगरेशन से मानों को ओवरराइड करने की अनुमति देता है?
ऐसा कुछ टूल में सामान्य है जैसे -g में nginx, -o ssh में, -c git में और कई और मुझे लगता है।

$ jshint -o "indent = 1" -o "-W034 = true" -o "globals.require = false" -o "globals.$ = null"

क्या किसी को पता है कि यह किस संस्करण में पेश किया गया था? 2.9.5?

कम से कम तब हम इस समय के लिए एक अस्थायी के रूप में डाउनग्रेड कर सकते थे। ठीक कर।

@jaredatch वापस https://github.com/AtomLinter/linter-jshint/pull/386 ( linter-jshint के v3.1.0 में जारी) में JSHint की बग्गी पॉइंट रिपोर्टिंग के लिए सभी वर्कअराउंड हटा दिए गए थे। उस चेक का पूरा बिंदु बगों को ढूंढना है जैसे कि लिंटर अमान्य बिंदुओं की रिपोर्ट कर रहा था ताकि इसे रिपोर्ट किया जा सके और लिंटर का उपयोग करके _everyone_ के लिए तय किया जा सके। (आखिरकार, यदि आप सटीक डेटा देने के लिए लिंटर पर भरोसा नहीं कर सकते हैं, तो इसका उपयोग क्यों करें?)

इस टैब बग को पेश किए जाने से पहले और प्रमुख पार्सर बग के बाद _may_ एक "स्वीट स्पॉट" हो, जिसके कारण उन वर्कअराउंड को पहले स्थान पर रखा गया था।

@Arcanemagus मिलाचा , वास्तव में अंतर्दृष्टि की सराहना करते हैं। बढ़िया काम करते रहो

यदि आपके पास linter-jshint रिपोर्ट करने के तरीके के बारे में एक बेहतर / अधिक उपयोगी संदेश विचार है, तो ये अमान्य बिंदु बेझिझक एक मुद्दा दर्ज करें / वहां एक पीआर जमा करें ।

कृपया अपने सभी समाधान पिछले कमबख्त 5 घंटे बिताएं, एक कमबख्त जावास्क्रिप्ट फ़ाइल को कमबख्त 2017 की कोशिश कर रहा है, मुझे यह समस्या क्यों हो रही है परमाणु मेरे कमबख्त जीवन का कमबख्त प्रतिबंध है आप अपने जीवन को ठीक करते हैं

और जो बकवास वास्तव में डब्ल्यूटीएफ को इंडेंट करने के लिए टैब का उपयोग नहीं करता है, आप लाखों लोगों के लिए एक टूटा हुआ पैकेज क्यों जारी करेंगे जो जेएस को लिंट करते हैं ???????

पीरियड्स का शाब्दिक उपयोग कौन नहीं करता है, आप इस टूटी-फूटी अंग्रेजी को लाखों लोगों के लिए क्यों जारी करते हैं जो कि अंग्रेजी है?

ओह। और बड़े अक्षरों में वर्ण वाक्य की शुरुआत में जाते हैं। हर जगह नहीं।

/ ट्रोल (श्रीमान, खुद की मदद नहीं कर सका)

खैर, मुझे लगता है कि समाधान कुछ समय के लिए इसे निष्क्रिय कर देगा...
वैसे भी लिंटर्स की जरूरत किसे है?

@jugglinmike क्या आप https://github.com/jshint/jshint/issues/3151#issuecomment -312512856 में सवालों के जवाब दे सकते हैं?

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

यह मुद्दा मेरे रडार से फिसल गया; मुझे इसके लिए खेद है, @Arcanemagus। दुर्भाग्य से, मुझे नहीं लगता कि मैं इस सप्ताह के अंत तक इस पर ध्यान देने योग्य हूं। क्या यह आपके काम आएगा?

@jugglinmike बेशक

मैंने मौजूदा मुद्दों के लिए कुछ जाँचें लगाईं, इसलिए अधिकांश भाग के लिए लोगों को पहले से ही खुले मुद्दों को यहाँ इंगित करने के लिए फ़िल्टर किया जा रहा है।

@cobexer मुझे वह विचार पसंद है क्योंकि हमें आगे बढ़ने का रास्ता देने के अलावा, यह
अन्य संदर्भों में लोगों के लिए उपयोगी हो सकता है। उसने कहा, मेरी जानकारी के लिए वहाँ
उस तरह की कार्यक्षमता के लिए कभी भी अनुरोध नहीं किया गया है, और मैं इसके लिए अनिच्छुक हूं
किसी भी नई सुविधा के लिए प्रतिबद्ध सिर्फ इसलिए कि यह अच्छा लगता है। और इसका कारण है
विशेष रूप से उस सुविधा को हतोत्साहित करें: "आरसी" फाइलों के बीच, इन-लाइन
कॉन्फ़िगरेशन, और Node.js API, उपयोगकर्ता जब चाहें तब ट्रिप हो जाते हैं
यह समझने के लिए कि दिया गया विकल्प सक्षम क्यों है। कॉन्फ़िगरेशन के लिए एक और वेक्टर जोड़ना
प्रबंधन इस समस्या को और बढ़ा देगा।

मुझे लगता है कि यहां सबसे व्यवहार्य समाधान मेरे पहले के आधार पर हैप्रस्ताव ।
हालांकि, हमें character संपत्ति को सार्वजनिक एपीआई के रूप में मानना ​​होगा--एटम
संपादक प्लगइन स्वयं उस पर बनाया गया है, आखिरकार। तो उसे संशोधित करने के बजाय
संपत्ति और एक नए को परिभाषित करना जिसमें वर्तमान के शब्दार्थ हैं, मुझे लगता है
हमें character को यथावत छोड़ना होगा और अनुरोधित डेटा को नए पर प्रदर्शित करना होगा
संपत्ति का नाम।

मैंने इस सप्ताह के अंत में नई सुविधा पर काम शुरू किया। मैं प्रतिगमन से बचना चाहता हूँ
सभी लागतें, इसलिए हमें लगातार जोर देने के लिए परियोजना के परीक्षणों का विस्तार करने की आवश्यकता है
अपेक्षित वर्ण संख्या। जैसे ही मैंने यह करना शुरू किया, मुझे इसमें कमियां मिलीं
परीक्षण सूट जिसे पहले संबोधित करने की आवश्यकता है। gh-3174 की ओर पहला कदम है
वह लक्ष्य।

यह सब कहना है: ऐसा लगता है कि इसमें कुछ समय लग सकता है। @Arcanemagus मुझे पता है
आप इस बग का खामियाजा भुगत रहे हैं, और मैं इसे एक के रूप में लेने की सराहना करता हूं
JSHint को बेहतर देखने का अवसर। मुझे यह भी पसंद है कि आप इसमें रुचि रखते हैं
ऐसे किसी भी समाधान से बचना जो आपके उपयोगकर्ताओं के लिए पारदर्शी न हो। संक्षेप में
शब्द, क्या आपके पास कोई भाग्य है जो उपयोगकर्ताओं को indent विकल्प को 1 पर सेट करने की अनुशंसा करता है?
यह आदर्श नहीं है, लेकिन ऐसा करने से उनकी अल्पकालिक समस्या एक तरह से हल हो जानी चाहिए
यह पैच अप होने के बाद भी काम करता रहेगा।

@Arcanemagus मुझे पता है कि आप इस बग का खामियाजा भुगत रहे हैं, और मैं इसे एक के रूप में लेने की सराहना करता हूं
JSHint को बेहतर देखने का अवसर।

एटम के लिए अधिकांश लिंटर प्रदाताओं द्वारा उपयोग की जाने वाली जेनेरिक लाइब्रेरी में चेक जोड़ने का पूरा कारण यह है कि इस तरह की बग स्रोत लिंटर्स में पकड़ी जा सकती है और रिपोर्ट/निश्चित । यह वास्तव में पहले पकड़ा गया होगा, लेकिन linter-jshint में एक चेक था जो मूल रूप से _way_ से सभी अमान्य बिंदु त्रुटियों को छुपाता था जब अधिकांश दस्तावेज़ों पर पार्सर पूरी तरह विफल हो गया था और हमारे पास इसकी रिपोर्ट करने का एक अच्छा तरीका नहीं था उपयोगकर्ताओं या डी-डुप्लिकेटिंग समस्या रिपोर्ट के लिए। अब जब उनका पता लगा लिया गया है तो मैंने उस चेक को हटा दिया है और यहां हम हैं।

मुझे यह भी पसंद है कि आप किसी ऐसे समाधान से बचने में रुचि रखते हैं जो आपके उपयोगकर्ताओं के लिए पारदर्शी न हो।

असल में, इस विशेष मुद्दे के लिए एक पारदर्शी समाधान ठीक होगा, मुझे नहीं पता कि indent सेटिंग को 1 पर मजबूर करने से वे जो परिणाम प्राप्त करेंगे उसकी तुलना में वे जो परिणाम देख रहे हैं उन्हें बदल देगा। jshint स्वयं चल रहा है।

अल्पावधि में, क्या आपके पास कोई भाग्य है जो उपयोगकर्ताओं को इंडेंट विकल्प को 1 पर सेट करने की सलाह देता है?

यह... एक उत्कृष्ट विचार है कि मैंने वास्तव में उन मुद्दों पर पोस्ट करने के बारे में नहीं सोचा था। मैं यह उल्लेख करना सुनिश्चित करूँगा कि मुद्दों पर अधिक टिप्पणी में!

ओह, हाँ, .jshintrc में 'इंडेंट' गुण को 1 पर सेट करना एक उत्कृष्ट समाधान है। काश मुझे इसके बारे में जल्दी पता होता। मेरे पास पहले से .jshintrc है क्योंकि jshint es6 के लिए डिफ़ॉल्ट नहीं है, इसलिए इसे जोड़ना आसान है। मैं

यह सुनकर खुशी हुई, @ tustin2121!

अद्यतन: मैं अभी भी प्रतिगमन से बचने के लिए JSHint परीक्षण सूट को किनारे करने के लिए काम कर रहा हूं। उस प्रयास में नवीनतम gh-3176 है।

इस मुद्दे पर कोई अपडेट?

ऐसा लगता है कि अब हम इस बग से कम से कम 40 अलग-अलग नियमों को प्रभावित कर रहे हैं .

इस पर कोई अपडेट, यह कुछ महीनों के बाद से मुझे यह त्रुटियां हो रही हैं, मुझे पागल कर रही है :)

हाय @xcrap! मैंने अभी टिप्पणी सूत्र की जाँच की है, और ऐसा नहीं लगता कि किसी ने कोई अपडेट पोस्ट किया है। अच्छी खबर यह है कि JSHint एक ओपन सोर्स प्रोजेक्ट है, इसलिए यदि यह आपको तनाव दे रहा है तो आप समस्या को ठीक करने में मदद कर सकते हैं! क्या आप एक हाथ उधार देना चाहेंगे? मुझे आपको सलाह देने में खुशी होगी।

@jugglinmike मेरी इच्छा है! मैं हमेशा छोटी परियोजनाओं में कोशिश करता हूं लेकिन मेरे जेएस कौशल सुपर बेसिक और सीमित हैं :)

मैंने इस बग पर $50 का इनाम रखा है, अन्य लोग इस लिंक के माध्यम से इनाम बढ़ाने में योगदान दे सकते हैं: https://www.bountysource.com/issues/46533252-tab-indentation-breaks-multiple-rules

स्थानीय रूप से @tzvipm के कांटे को खींचना और jshint के पुनर्निर्माण ने मेरे लिए इस मुद्दे को ठीक करने का काम किया अगर कोई और इस पर ठोकर खा रहा है:

git clone https://github.com/tzvipm/jshint
git remote add upstream [email protected]:jshint/jshint.git
git fetch upstream
git stash
git merge upstream/master
git mergetool --tool=opendiff
npm run build