Tslint: يجب أن تقوم قاعدة الفاصلة المنقوطة بوضع علامة على الفاصلة المنقوطة غير الضرورية في وظائف أعضاء الفئة

تم إنشاؤها على ١١ أغسطس ٢٠١٦ · 8تعليقات · مصدر: palantir/tslint

قمنا بتغيير سلوك قاعدة الفاصلة المنقوطة للسماح بحذف الفاصلة المنقوطة اللاحقة بعد وظائف عضو الفئة المرتبطة:

class Foo {
  public renderFoo = () => {
    return null;
  }
   ^ right here
}

أقترح أننا في الواقع _ban_ فاصلة منقوطة في ذلك الموقع في TSLint 4.0. يمكننا دعم الوظيفة القديمة بخيار جديد allow-bound-class-methods .

Easy P2 Fixed Breaking Change Enhancement

مصدر

adidahiya

👎10 👍9 😕2

التعليق الأكثر فائدة

بالنسبة للأشخاص الذين يأتون إلى هذا الموضوع ويبحثون عن تكوين tslint الصحيح (كما فعلت أنا) - سيكون بناء الجملة الصحيح لهذا هو:
"فاصلة منقوطة": [true، "always"، "ignore-bound-class-Methods"]

انظر: https://palantir.github.io/tslint/rules/semicolon/

popstr في ١٨ أبريل ٢٠١٧

👍34 🎉2

ال 8 كومينتر

لست مقتنعًا بأن هذه بحاجة إلى 4.0 ، لكنني لن أؤذي الحصول عليها أيضًا. يبدو أنه خيار دقيق قليلاً ، لكنني لست مقتنعًا تمامًا أنه يستحق التعقيد الإضافي

jkillian في ٥ أكتوبر ٢٠١٦

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

ربما لا أفهم تمامًا الأساس المنطقي وراء ذلك ، ولكن على وجه الخصوص ، يبدو أن الفاصلة المنقوطة لعلامات tslint غير ضرورية في هذا الموضع (أي لا تقبل فقط تركها) تبدو مربكة.

mprobst في ١٣ ديسمبر ٢٠١٦

👍6

يمكنك استعادة السلوك القديم باستخدام الخيار ignore-bound-class-methods . أردنا أن نكون أكثر إبداء الرأي مع التكوين الافتراضي وأن نجعل طرق الفئات المقيدة تبدو مثل طرق الفئات العادية في معظم أكواد TS.

adidahiya في ١٣ ديسمبر ٢٠١٦

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

mprobst في ١٣ ديسمبر ٢٠١٦

👍4

حاولت أن أجعل هذا يتوقف عن إلقاء الأخطاء في انتليج

حاولت إزالة الفاصلة المنقوطة الموجودة خلف وظائف سهم الفئة ، ثم تلقيت هذا التحذير:

حاولت تمكين الخيار مثل هذا:

لكني حصلت على هذه النتيجة:

حاولت تمكين الخيار مثل هذا:

ولكن بعد ذلك يتوقف عن البحث عن الفواصل المنقوطة معًا.

إصدار tslint: 4.4.2
إصدار TS: 2.1.5
نسخة انتليج: 2016.3
نظام التشغيل: windows 10 x64

bertyhell في ١ فبراير ٢٠١٧

👍14

انظر: https://palantir.github.io/tslint/rules/semicolon/

popstr في ١٨ أبريل ٢٠١٧

👍34 🎉2

popstr نعم ، هذا ما أستخدمه أيضًا.

"semicolon": [true, "always", "ignore-bound-class-methods"]

robertpenner في ١٨ أبريل ٢٠١٧

👍3 😄1

لقد أنشأت # 3216 لأن التجاهل فكرة سيئة من حيث الاتساق ، والقاعدة بدون الخيار ignore-bound-class-methods لا تزال عربات التي تجرها الدواب كما اكتشفت.