Tslint: Aturan titik koma harus menandai titik koma yang tidak perlu pada fungsi anggota kelas

Dibuat pada 11 Agu 2016 · 8Komentar · Sumber: palantir/tslint

Kami mengubah perilaku aturan titik koma untuk mengizinkan penghilangan tanda titik koma setelah fungsi anggota kelas terikat:

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

Saya mengusulkan agar kita benar-benar _ban_ titik koma di lokasi itu di TSLint 4.0. Kami dapat mendukung fungsi lama dengan opsi baru allow-bound-class-methods .

Easy P2 Fixed Breaking Change Enhancement

Sumber

adidahiya

👎10 👍9 😕2

Komentar yang paling membantu

Untuk orang-orang yang datang ke utas ini mencari konfigurasi tslint yang benar (seperti yang saya lakukan) - sintaks yang benar untuk ini adalah:
"titik koma": [benar, "selalu", "abaikan-terikat-kelas-metode"]

Lihat: https://palantir.github.io/tslint/rules/semicolon/

popstr pada 18 Apr 2017

👍34 🎉2

Semua 8 komentar

Saya tidak yakin ini adalah kebutuhan untuk 4.0, tetapi tidak ada salahnya untuk memasukkannya juga. Itu memang tampak seperti opsi yang sebentar, saya tidak sepenuhnya yakin itu sepadan dengan kerumitan tambahannya

jkillian pada 5 Okt 2016

Sama seperti item umpan balik (tidak yakin apakah ini tempat yang tepat?), Kami baru saja menemukan ini, dan sangat terkejut dengan ini.

Saya mungkin tidak begitu memahami alasan di balik ini, tetapi khususnya bahwa tslint menandai titik koma sebagai tidak perlu dalam posisi ini (yaitu tidak hanya menerima meninggalkannya) tampaknya membingungkan.

mprobst pada 13 Des 2016

👍6

Anda bisa mendapatkan kembali perilaku lama dengan opsi ignore-bound-class-methods . Kami ingin sedikit lebih berpendirian dengan konfigurasi default dan membuat metode kelas terikat terlihat seperti metode kelas biasa di sebagian besar kode TS.

adidahiya pada 13 Des 2016

Ya, saya pikir begitu banyak. Saya berharap tslint untuk out-of-the-box membutuhkan titik koma itu, lebih banyak berbuat salah di sisi konsistensi, atau mengabaikan masalah, tetapi menambahkan opsi itu kembali berfungsi untuk kami.

mprobst pada 13 Des 2016

👍4

Saya mencoba untuk mendapatkan ini untuk berhenti melempar kesalahan di intellij

Saya mencoba menghapus titik koma di belakang fungsi panah kelas, lalu saya mendapatkan peringatan ini:

Saya mencoba mengaktifkan opsi seperti ini:

Tetapi saya mendapatkan hasil ini:

Saya mencoba mengaktifkan opsi seperti ini:

Tapi kemudian berhenti memeriksa titik koma secara bersamaan.

versi tslint: 4.4.2
versi ts: 2.1.5
versi intellij: 2016.3
OS: windows 10x64

bertyhell pada 1 Feb 2017

👍14

Lihat: https://palantir.github.io/tslint/rules/semicolon/

popstr pada 18 Apr 2017

👍34 🎉2

@popstr Ya, itulah yang saya gunakan juga.

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

robertpenner pada 18 Apr 2017

👍3 😄1

Saya telah membuat #3216 karena mengabaikan adalah ide yang buruk dalam hal konsistensi, dan aturan tanpa opsi ignore-bound-class-methods masih bermasalah seperti yang saya ketahui.