Openlibrary: دعونا نفحص جافا سكريبت لدينا مع eslint!

تم إنشاؤها على ١٧ أكتوبر ٢٠١٨ · 5تعليقات · مصدر: internetarchive/openlibrary

يمنحنا Stylelint الحماية من مشكلات CSS المختلفة. لنفعل الشيء نفسه مع JS.
eslint هي الأكثر نضجًا وتوفر قائمة بالقواعد التي يمكننا الاستفادة منها إذا رأينا الحاجة.

npm install eslint --save-dev

في package.json ، أضف أمرًا جديدًا

"lint:js": "eslint openlibrary/plugins/openlibrary/js/"

أضف ملف .eslintrc.json

{
    "root": true,
    "env": {},
    "globals": {
    },
    "rules": {
        "no-extra-semi": "warn"
    }
}

سنقوم بتعيين الأخطاء للتحذير في الوقت الحالي وبمجرد دمج التكوين ، يجب أن ننسق مع شخص ما لتشغيل الإصلاحات والدمج لتجنب أي تعارضات في الدمج!

[] npm run lint: js ستظهر لي التحذيرات المتعلقة بقاعدة "no-extra-sem".
[] يحدث "npm run lint: js " عند تشغيل "npm run test"
[] لا تقم بإصلاح أي أخطاء كجزء من هذا التغيير!

Good First Issue Linting JavaScript 1 hacktoberfest

مصدر

jdlrobson

ال 5 كومينتر

مرحبا. هل يمكنك تكليفي لحل هذه المشكلة؟

0x15F9 في ١٨ أكتوبر ٢٠١٨

سؤال: لماذا لا تستخدم أحد الإعدادات المسبقة لـ eslint ؟ مثل https://standardjs.com أو https://github.com/airbnb/javascript

tamtamchik في ٢٠ أكتوبر ٢٠١٨

أرغب في ذلك على المدى الطويل ولكن جافا سكريبت في حالة سيئة للغاية ، فإن تطبيق مثل هذا الإعداد المسبق سيكون مربكًا كثيرًا (مراجعة الكود ، تعارضات إعادة القاعدة) والعمل اليدوي (لا يمكن إصلاح معظم الإعداد المسبق تلقائيًا). أعتقد أن إضافة القواعد يدويًا تساعدنا على مهاجمتها شيئًا فشيئًا بطريقة متماسكة وتعليمية (نتعرف على المشكلات في الكود لدينا شيئًا فشيئًا).

لقد اعتقدت أنه يمكننا استخدام الفحص لحل المشكلات الأكثر شيوعًا وإلحاحًا (النسق التي يمكن أن تسبب أخطاء في التعليمات البرمجية) ثم في وقت فراغنا نطبق إعدادًا مسبقًا (من اختيارنا) لاحقًا على الطريق.

هل لهذا معنى؟

jdlrobson في ٢٠ أكتوبر ٢٠١٨

👍1

بالتأكيد ، إنه كذلك. يمكن أن يكون شكل الكود سيئًا للغاية.

عندما يتم دمج هذا PR ، يمكنني إصلاح بعض المشكلات مع الفحص ، فقط قم بتسمية em.
سيبدأ بـ no-extra-semi على سبيل المثال ؛)

tamtamchik في ٢٠ أكتوبر ٢٠١٨

👍1

tamtamchik يمكنني إنشاء مشكلة لاحقًا ولكن أي شيء موجود ضمن "الأخطاء المحتملة" وقابل للإصلاح التلقائي يعد لعبة عادلة.

يجب أن ننسق مع mekarpeles لتشغيل وتنفيذ عمليات الإصلاح التلقائي وارتكاب خطأ eslint بدلاً من التحذير.

أريد أن نتخلص من الفاكهة المعلقة المنخفضة!

jdlrobson في ٢١ أكتوبر ٢٠١٨

🎉1