Less.js: [Tugas Pemeliharaan] - Tinjau pengaturan ESLint dan perbaiki masalah linting

Dibuat pada 21 Jun 2018 · 6Komentar · Sumber: less/less.js

Beberapa waktu yang lalu saya mengalihkan proyek dari JSHint ke ESLint (yang terakhir lebih terawat dan menawarkan lebih banyak fitur). Namun, saya menetapkan banyak pengaturan sebagai peringatan karena saya tidak punya waktu untuk mengatasi masalah potensial. Saya juga tidak punya waktu untuk men-tweak linting untuk pengaturan yang membuat kode lebih mudah dipelihara.

Jadi, tugas dasarnya adalah melihat peringatan ESLint, dan melihat apa yang dapat ditangani dengan aman. (Misalnya, banyak peringatan kesetaraan == mungkin tidak "dapat diperbaiki" ke === tanpa pengujian untuk setiap nilai yang melewati pemeriksaan itu.) Akan sangat membantu juga untuk menyetel peringatan untuk hal-hal yang harus ditangani di masa depan / sedang berlangsung, seperti menegakkan dokumentasi JSDoc.

good first issue medium priority up-for-grabs

Sumber

matthew-dean

Semua 6 komentar

Halo, saya ingin mengambil ini!

umuur pada 18 Jan 2021

@umuur Ayo!

matthew-dean pada 19 Jan 2021

👍1

@matthew-dean Apakah kita ingin mempertahankan konfigurasi eslint saat ini?
Saya tidak yakin memiliki TypeScript sebagai parser dan plugin diperlukan.

umuur pada 19 Jan 2021

@umuur

Apakah kita ingin mempertahankan konfigurasi eslint saat ini?

Tergantung. Jika Anda ingin membuat perubahan yang tidak mengubah pemformatan kode, tidak apa-apa. Tetapi sejauh peringatan, IMO itu semua adalah peringatan yang valid dan kami ingin mengatasinya, atau meninggalkan komentar kode tentang eslint yang dinonaktifkan dan (yang paling penting ) mengapa itu tidak valid di sana.

Apakah kita ingin mempertahankan konfigurasi eslint saat ini?

Meskipun basis kode belum menggunakan TypeScript, menurut pengalaman saya, ia melakukan pekerjaan yang lebih baik daripada Babel. Dalam hal ESLint.... ya Anda benar itu tidak diperlukan secara teknis sampai ada kode di TypeScript. Saya mungkin melakukan itu untuk mengantisipasi mengubah basis kode menjadi TS, tetapi sekarang saya tidak tahu apakah/kapan itu akan terjadi.

matthew-dean pada 20 Jan 2021

👍1

@umuur

Omong-omong, di mana pun Anda dapat menambahkan komentar JSDoc dengan tipe yang tepat pada params, silakan tambahkan!

matthew-dean pada 20 Jan 2021

@matthew-dean terima kasih atas komentar terperincinya! Akan memberi tahu Anda tentang pembaruan.

umuur pada 20 Jan 2021

👍1

Apakah halaman ini membantu?

0 / 5 - 0 peringkat