Less.js: [مهمة الصيانة] - مراجعة إعدادات ESLint وإصلاح مشكلات الفحص

تم إنشاؤها على ٢١ يونيو ٢٠١٨ · 6تعليقات · مصدر: less/less.js

منذ فترة ، قمت بتحويل المشروع من JSHint إلى ESLint (هذا الأخير يتم صيانته بشكل أفضل ويقدم المزيد من الميزات). ومع ذلك ، فقد قمت بتعيين العديد من الإعدادات كتحذيرات لأنه لم يكن لدي الوقت لمعالجة المشكلات المحتملة. كما لم يكن لدي الوقت الكافي لتعديل الإعدادات التي تجعل الكود أكثر قابلية للصيانة.

لذا ، ستكون المهمة في الأساس هي إلقاء نظرة على تحذيرات ESLint ، ومعرفة ما يمكن معالجته بأمان. (على سبيل المثال ، قد لا تكون الكثير من تحذيرات المساواة == "قابلة للإصلاح" إلى === بدون اختبار كل قيمة تمر عبر هذا الفحص.) سيكون من المفيد أيضًا تعيين تحذيرات لـ الأشياء التي يجب معالجتها في المستقبل / الجارية ، مثل إنفاذ وثائق JSDoc.

good first issue medium priority up-for-grabs

مصدر

matthew-dean

ال 6 كومينتر

مرحبًا ، أود الحصول على هذا!

umuur في ١٨ يناير ٢٠٢١

umuur اذهب لذلك!

matthew-dean في ١٩ يناير ٢٠٢١

👍1

@ ماثيو دين هل نريد الحفاظ على تكوين eslint الحالي؟
لست متأكدًا من أن وجود TypeScript كمحلل ومكوِّن إضافي ضروري.

umuur في ١٩ يناير ٢٠٢١

umuur

هل نريد الاحتفاظ بتكوين eslint الحالي؟

هذا يعتمد. إذا كنت تريد إجراء تعديلات لا تغير تنسيق الكود ، فلا بأس بذلك. ولكن فيما يتعلق بالتحذيرات ، IMO هذه كلها تحذيرات صالحة ونريد إما معالجتها ، أو ترك تعليق رمز حول eslint المعطل و ( الأهم من ذلك ) سبب عدم صلاحيتها هناك.

هل نريد الاحتفاظ بتكوين eslint الحالي؟

على الرغم من أن مصدر الشفرة لا يستخدم TypeScript حتى الآن ، فمن واقع خبرتي ، فإنه يقوم بعمل نقل أفضل من Babel. فيما يتعلق بـ ESLint .... 🤔 نعم أنت على حق ، فهي ليست ضرورية من الناحية الفنية حتى يكون هناك رمز في TypeScript. ربما فعلت ذلك تحسبًا لتحويل مصدر الشفرة إلى TS ، لكنني الآن لا أعرف ما إذا كان / متى سيحدث ذلك.

matthew-dean في ٢٠ يناير ٢٠٢١

👍1

umuur

بالمناسبة ، في أي مكان حيث يمكنك إضافة تعليقات JSDoc بأنواع مناسبة على المعلمات ، يرجى إضافتها!

matthew-dean في ٢٠ يناير ٢٠٢١

@ ماثيو دين أشكركم على التعليقات التفصيلية! سوف نعلمك عن التحديثات.

umuur في ٢٠ يناير ٢٠٢١

👍1

هل كانت هذه الصفحة مفيدة؟

0 / 5 - 0 التقييمات