Faraday: रूबोकॉप क्वेस्ट मुद्दा

को निर्मित 20 फ़र॰ 2019 · 5टिप्पणियाँ · स्रोत: lostisland/faraday

बुनियादी जानकारी

हम रूबोकॉप का उपयोग करके पूरे कोडबेस को लाइन कर रहे हैं।

जब हमने शुरुआत की .rubocop_todo.yml तब यह

यदि आप इनमें से किसी एक को ठीक करने के लिए पीआर बनाते हैं, तो रूबोकॉप में --only विकल्प का उपयोग करें, ताकि 1 प्रकार के सुधार पर प्रयास केंद्रित किया जा सके। इस तरह के कोड की समीक्षा करते समय अभिभूत होना इतना आसान है। (यदि आप इससे थोड़ा अधिक ठीक करते हैं, तो कोई बड़ी बात नहीं है। यह सभी परिवर्तनों को समीक्षा योग्य रखने के बारे में है।)

# edit .rubocop.yml, commenting out the first line -
#   this removes the "ignore the TODOs" setting

# Auto-correct with only 1 rule
rubocop -a --only Name/OfTheRuleYouAreFixing --auto-correct

# Re-generate the config file
rubocop --auto-gen-config

# Revert line 3 in .rubocop_todo.yml
# This will avoid PR conflicts

# Inspect to see if it looks OK
git diff

# Re-set the TODO list
git checkout .rubocop.yml

# Add and commit the change
git add .
git commit -m"chore: RuboCop lint Name/OfTheRuleYouAreFixing"

एक कार्यप्रवाह

मुदे का विवरण

[x] बंडलर/आदेशित रत्न #856
[x] लेआउट/एक्सेसमोडिफायर इंडेंटेशन #862
[x] लेआउट/एलाइनहैश #863
[x] लेआउट/संरेखण पैरामीटर #867
[x] लेआउट/ब्लॉकएंडन्यूलाइन
[x] लेआउट/केस इंडेंटेशन #८६४
[x] लेआउट/टिप्पणी इंडेंटेशन #864
[x] लेआउट/डॉटपोजिशन #880
[x] लेआउट/अन्य संरेखण #८७१
[x] लेआउट/EmptyLineAfterGuardClause #860
[x] लेआउट/खाली लाइन के बीच डिफ्स #860
[x] लेआउट/खाली रेखाएं #८६०
[x] लेआउट/EmptyLinesAroundAccessModifier #८८१
[x] लेआउट/एम्प्टीलाइन्सअराउंडक्लासबॉडी #882
[x] लेआउट/एंड एलाइनमेंट #887
[x] लेआउट/अतिरिक्त स्पेसिंग #884
[x] लेआउट/इंडेंटअरे #890
[x] लेआउट/इंडेंटहैश #890
[x] लेआउट/इंडेंटेशनविड्थ #864
[x] लेआउट/लीडिंग कमेंटस्पेस #860
[x] लेआउट/मल्टीलाइनब्लॉकलेआउट
[x] लेआउट/मल्टीलाइनमेथोडकॉल इंडेंटेशन #864
[x] लेआउट/मल्टीलाइनऑपरेशन इंडेंटेशन #864
[x] लेआउट/RescueEnsureAlignment #890
[x] लेआउट/स्पेसआफ्टरकॉमा #884
[x] लेआउट/स्पेसअराउंडएक्वाल्सइनपैरामीटरडिफॉल्ट #884
[x] लेआउट/स्पेसअराउंडऑपरेटर्स #884
[x] लेआउट/स्पेस बिफोरब्लॉकब्रेसेस #884
[x] लेआउट/स्पेसइनसाइडअरेलिटरलब्रैकेट्स #884
[x] लेआउट/स्पेसइनसाइडब्लॉकब्रेसेस #884
[x] लेआउट/स्पेसइनसाइडहैशलिटरलब्रेसेस #884
[x] लेआउट/स्पेसइनसाइडपेरेंस #८८४
[x] लेआउट/स्पेसइनसाइडपरसेंट लिटरल डिलीमीटर #884
[x] लेआउट/ट्रेलिंगब्लैंकलाइन #872
[x] लेआउट/ट्रेलिंगव्हाइटस्पेस
[x] लिंट/असाइनमेंटइनकंडीशन #874
[x] लिंट/हैंडल एक्सेप्शन #896
[x] लिंट/रिटर्नइनवोइड कॉन्टेक्स्ट #896
[x] लिंट/स्ट्रिंग कन्वर्जन इनइंटरपोलेशन #891
[x] लिंट/अननीडेड रिक्वायरमेंट स्टेटमेंट #८९६
[x] एक प्रकार का वृक्ष/अप्रयुक्तब्लॉकअर्ग्यूमेंट #896
[x] एक प्रकार का वृक्ष/अप्रयुक्त विधि आर्ग्यूमेंट #896
[x] लिंट/बेकार असाइनमेंट #८९१
[x] एक प्रकार का वृक्ष/शून्य #896
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/एबीसी आकार~
[x] मेट्रिक्स/ब्लॉकलेंथ #883 #927
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/कक्षा लम्बाई #944~
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/साइक्लोमैटिक कॉम्पलेक्सिटी~
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/विधि लंबाई~
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/मॉड्यूल लंबाई~
[x] _अनिश्चित काल के लिए स्थगित_ ~ मेट्रिक्स/अनुमानित जटिलता~
[x] नामकरण/ConstantName #897
[x] नेमिंग/मेमोइज्ड इंस्टेंस वेरिएबलनाम #891
[x] नामकरण/विधेयनाम #897
[x] नामकरण/असंचारी पद्धतिपरमनाम #897
[x] प्रदर्शन/अनावश्यकब्लॉककॉल #८९७
[x] प्रदर्शन/स्ट्रिंग रिप्लेसमेंट #897
[x] स्टाइल/एक्सेस मॉडिफायर डिक्लेरेशन #897
[x] शैली/उपनाम #897
[x] शैली/औरया #897
[x] स्टाइल/ब्लॉकडिलीमीटर #906
[x] स्टाइल/ब्रेसेसअराउंडहैशपैरामीटर #907
[x] शैली/मामला समानता #902
[x] Style/ClassAndModuleChildren #902
[x] स्टाइल/क्लासचेक #902
[x] स्टाइल/क्लासवार्स #933
[x] Style/ColonMethodCall #894
[x] शैली/टिप्पणीएनोटेशन #८७६
[x] स्टाइल/सशर्त असाइनमेंट #९०२
[x] शैली/दस्तावेज़ीकरण
[x] स्टाइल/डबल नेगेशन #९२२
[x] शैली/प्रत्येकविथऑब्जेक्ट #901
[x] शैली/खाली विधि #912
[x] शैली/विस्तारपथ आर्ग्यूमेंट्स #901
[x] स्टाइल/फॉर्मेटस्ट्रिंग #९०१
[x] स्टाइल/फॉर्मेटस्ट्रिंगटोकन #९१४
[x] स्टाइल/फ्रोजनस्ट्रिंग लिटरल कमेंट #868
[x] स्टाइल/ग्लोबल वार्स #943
[x] स्टाइल/गार्ड क्लॉज #931
[x] स्टाइल/हैशसिंटैक्स #878
[x] स्टाइल/इफइनसाइड एल्स #915
[x] स्टाइल/इफअनलेसमोडिफायर #926
[x] स्टाइल/लैम्ब्डा #901
[x] स्टाइल/विधि मिसिंगसुपर #९२८
[x] स्टाइल/मिसिंग रिस्पॉन्ड टू मिसिंग #930
[x] स्टाइल/मॉड्यूल फंक्शन #925
[x] स्टाइल/मल्टीलाइनआईएफमोडिफायर #९२४
[x] स्टाइल/मल्टीलाइन टर्नरीऑपरेटर #९१३
[x] शैली/एकाधिक तुलना #९२३
[x] स्टाइल/म्यूटेबल कॉन्स्टेंट #925
[x] शैली/नकारात्मक यदि #901
[x] स्टाइल/नेस्टेडपेरेंटेसाइज्ड कॉल्स #901
[x] स्टाइल/अगला #901
[x] शैली/शून्य तुलना #921
[x] शैली/नहीं #901
[x] शैली/संख्यात्मक प्रेडिकेट #921
[x] स्टाइल/समानांतर असाइनमेंट #920
[x] शैली/कोष्ठक आस-पास की स्थिति #९२०
[x] शैली/प्रतिशतसाहित्यिक सीमांकक #905
[x] स्टाइल/पर्लबैकरेफ्स #920
[x] स्टाइल/पसंदीदा हैशमेथड्स #920
[x] स्टाइल/प्रोक #920
[x] स्टाइल/RaiseArgs #910
[x] शैली/अनावश्यक अपवाद #895
[x] शैली/अनावश्यक कोष्ठक #८९५
[x] स्टाइल/अनावश्यक रिटर्न #895
[x] शैली/अनावश्यक स्वयं #८९५
[x] स्टाइल/रेगेक्सपलिटरल #९१०
[x] शैली/बचाव मानक त्रुटि #910
[x] शैली/सुरक्षित नेविगेशन #909
[x] शैली/अर्धविराम #908
[x] स्टाइल/सिंगललाइनमेथड्स #909
[x] स्टाइल/स्पेशल ग्लोबल वार्स #909
[x] स्टाइल/StderrPuts #908
[x] स्टाइल/स्ट्रिंग लिटरल्स #874
[x] स्टाइल/स्ट्रिंग लिटरल्सइनइंटरपोलेशन #909
[x] स्टाइल/स्ट्रक्चर इनहेरिटेंस #९०४
[x] शैली/प्रतीक सरणी #898
[x] शैली/SymbolProc #८९८
[x] स्टाइल/टर्नरीपैरेंटेस #898
[x] स्टाइल/ट्रेलिंगCommaInArrayLiteral #859
[x] स्टाइल/ट्रेलिंगCommaInHashLiteral #859
[x] स्टाइल/ट्रिविअल एक्सेसर्स #898
[x] शैली/अन्यथा #898
[x] स्टाइल/वर्डअरे #893
[x] स्टाइल/योडा कंडीशन #898
[x] शैली/शून्य लम्बाई प्रेडिकेट #858
[x] मेट्रिक्स/लाइनलेंथ #938

help wanted

स्रोत

olleolleolle

👍1

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

सबको धन्यवाद! मैं

olleolleolle 8 अप्रैल 2019

🎉3

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

इस टिप्पणी को हटा रहा है। मेरा पिछला दावा गलत था। उल्लंघन के लिए जाँच करते समय भविष्य के देवों को ध्यान देना चाहिए कि .rubocop_todo.yml फ़ाइल।

jherdman 25 फ़र॰ 2019

वर्तमान मास्टर रेफरी में कुछ रूबोकॉप उल्लंघन हैं जिन्हें सीआई द्वारा पकड़ा नहीं जा रहा है। वे मुद्दे https://github.com/lostisland/faraday/pull/870 और https://github.com/lostisland/faraday/pull/871 में तय किए गए हैं

साथ ही, यदि पीआर को [RuboCop] fix Layout/TrailingBlankLines जैसा कुछ नाम दिया गया है, तो उन्हें उपरोक्त चेकलिस्ट में जोड़ना बहुत आसान हो जाएगा।

technoweenie 25 फ़र॰ 2019

मैंने Style/GlobalVars क्योंकि एकमात्र अपराधी script/generate_certs । एक बार पुराने एकीकरण परीक्षण पूरी तरह से पोर्ट हो जाने के बाद, एकीकरण सूट के लिए पुरानी स्क्रिप्ट जा सकती है, और यह अपराध जादुई रूप से तय हो जाएगा। कहीं और जोड़े गए किसी भी अन्य वैश्विक युद्ध को रूबोकॉप अपराधों के रूप में चिह्नित करना जारी रखना चाहिए।

technoweenie 5 मार्च 2019

👍2

मैंने Style/Globalvars तय करने से पहले इस पोस्ट को नहीं देखा था: man_shrugging:

Insti 10 मार्च 2019