Tslint: تحسينات قاعدة المسافة البادئة

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

على غرار ESLint ، يجب أن نكون قادرين على فرض مسافة بادئة 2 أو 4 مسافات عبر خيارات مثل:

  • [x] [true, "spaces", 2]
  • [x] [true, "spaces", 4]

بالإضافة إلى ذلك ، سيكون من الجيد أن يكون لديك خيار "detect" (كما هو مقترح في # 122) والذي يحاول تخمين نمط المسافة البادئة في ملف وفرض الاتساق:

  • [] [true, "detect"]
P1 Fixed Enhancement

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

تم دمج @ dimitriy-k # 2723 للتو ، وسيكون في الإصدار التالي.

ال 37 كومينتر

+1

+1 لفرض عدد الفراغات.

+1

هل سيعمل هذا فقط إذا كانت المسافة البادئة قابلة للقسمة على عدد المسافات المكونة؟

{
    theObject.something()
             .more(); // 13 spaces.
}

لن يتم دعم طريقة (محاذاة الفترات) أعلاه؟

@ glen-84 أرى احتمالين:

  • فحص المسافة البادئة سطرًا بسطر (حيث لن يُسمح بمثالك
  • بيان بيان التحقق من المسافة البادئة. نظرًا لأن theObject.something().more() مثالك عبارة عن بيان واحد ، فإنه سيتحقق فقط من مستوى المسافة البادئة في بدايته.

أعتقد أن كلاهما له إيجابيات وسلبيات ، ولكن ربما يفضل الثاني هنا.

: +1:

+1
JKillian -> سأصوت للخيار رقم 2 حيث يتم فحص البيانات ، وليس كل سطر.

+1 الخيار 2 ، فحص البيان على حدة

JKillian شكرا. الخيار الثاني يبدو جيدًا. =)

تحرير: لقد غيرت رأيي .

+1 الخيار 1

طريقة جيدة هي ESLINT http://eslint.org/docs/rules/indent

+9000 SSJ3
من فضلكم ، يا رفاق ، قم بتنفيذ خيار المسافة البادئة 4 مسافات في أسرع وقت ممكن. لقد بدأت البرمجة في الكتابة المطبوعة كثيرًا ، ويبدو أن المسافة البادئة الافتراضية المكونة من مسافتين سيئة للغاية بالنسبة لي. بالنسبة لبعض الأشخاص ، من الصعب حقًا رؤية مستوى مسافة بادئة لبعض التعليمات البرمجية بمسافتين فقط ...

radarsu قد ترغب في تجربة استخدام .editorconfig في الوقت الحالي.

radarsu في الوقت الحالي ،

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

من الصعب تحديد مدى صعوبة الأمر - أتوقع أنه لن يكون بسيطًا على الأقل هههه. بالتأكيد اذهب لذلك!

mcriss @ Strike-sbolel الرجاء التوقف عن إضافة تعليقات +1 .

يبدو أنه يتم بذل جهود مماثلة هنا https://github.com/buzinas/tslint-eslint-rules/issues/74

لقد غيرت رأيي ، لأن الخيار 2 سيسمح برمز مثل:

{
    theObject.something()
             .more() // 13 spaces.
           .wrong()
                    .moreWrong();
}

في عالم مثالي ، سنستخدم الخيار 1 مع إعداد للسماح (أو لفرض) وضع الخطوط "الملتفة" أسفل الفترة الأخيرة في السطر السابق.

{
    theObject.something()
             .more()  // 13 spaces.
             .right() // Must be the same as above now, because the previous line used this indentation mode.
             .moreRight();
}

أو:

{
    theObject.something()
        .more() // (4 x 2) spaces.
        .right()
        .moreRight();
}

هل يمكننا إصدار تحديث من فضلك؟

owais لم يتم تنفيذ التحسينات في TSLint الأساسية حتى الآن. هذا الرابط عبارة عن ريبو خارجي بقواعد مخصصة.

أوه. لم أدرك أنني كنت أبحث في ريبو آخر. :) شكرا على الرابط. أعتقد أنني قد أستخدم ذلك.

adidahiya هل ما زال هذا شيئًا رائعًا ، وهل يستحق أن

سيسعدpudgeball بالتأكيد بقبول العلاقات العامة (كما هو موضح بواسطة الملصق)

لاحظ أن # 2723 لا يصلح هذه المشكلة. يعمل على إصلاح علامات التبويب -> مسافات س ومسافات س -> علامات تبويب ، ولكنه لا يصلح مسافات س -> مسافات ص

لا تزال هناك تحديثات على حجم المسافة البادئة؟ ... الطلب من عام 2015

تم دمج @ dimitriy-k # 2723 للتو ، وسيكون في الإصدار التالي.

ماذا ستسمى هذه القاعدة؟ 😄

dolanmiu ليست قاعدة جديدة. يتم تنفيذه تمامًا كما هو مذكور في وصف الإصدار الأصلي. فقط انقر للوصول إلى العلاقات العامة التي قمت بربطها.

adidahiya أنا أستخدم أحدث إصدار من tslint 5.3.2

وباستخدام ما يلي:

    "indent": [
      true,
      "spaces",
      4
    ],

لدي ملف مكون من مسافتين ، لكنه لم يكتشفه! هل أفعل شيئًا خاطئًا أم أن هناك خطأ؟

dolanmiu نعم يبدو أنه إصدار جديد ، كما أنني لاحظت أن # 2723 لم يعمل كما توقعت. لقد قدمت https://github.com/palantir/tslint/issues/2814

أي تحديثات؟

"indent": [true, "spaces", 4] لا يعمل لدي

@ liul85 جرب تثبيت قواعد tslint-eslint واستخدم قاعدة terIndent

leocaseiro شكرا للمشاركة ، سأحاول.

لا يزال لا يعمل في tslint 5.11.0

لا يزال "indent": [true, "spaces", 4] لا يعمل ...

هل كانت هذه الصفحة مفيدة؟
0 / 5 - 0 التقييمات

القضايا ذات الصلة

allbto picture allbto  ·  3تعليقات

cateyes99 picture cateyes99  ·  3تعليقات

DanielKucal picture DanielKucal  ·  3تعليقات

CSchulz picture CSchulz  ·  3تعليقات

avanderhoorn picture avanderhoorn  ·  3تعليقات