Tslint: Tambahkan opsi jam tangan

Dibuat pada 23 Agu 2017  ·  14Komentar  ·  Sumber: palantir/tslint

Saya belum menemukan masalah dengan permintaan seperti itu, jadi saya membuatnya

Declined Feature Request

Komentar yang paling membantu

Saya tidak setuju bahwa ini harus ditutup, dan setidaknya harus tetap pada peta jalan bahkan jika fokus saat ini ada di tempat lain. Menonton tampaknya terlalu rumit dibandingkan dengan tsc -w . tsc

Semua 14 komentar

Saya ingin mengatasi masalah ini karena tidak ada aktivitas sejak 24 Agustus dan saya pikir itu penting.

Saya menggunakan nodemon untuk melihat perubahan file dan menjalankan tslint dalam skrip npm. Salah satu contoh https://github.com/ryancat/create-ts-library/blob/master/package.json#L9

Anda tidak perlu keluar dengan kode kesalahan 2 untuk menghindari nodemon keluar.

Hanya ingin menekankan perlunya fitur ini.

Saya ingin fitur ini diterapkan

Aku demam. Dan satu-satunya resep adalah lebih banyak --watch .

ini adalah fitur _sangat sulit_ untuk diterapkan karena kemungkinan memerlukan perancangan ulang secara radikal untuk mendukung pemutaran ulang inkremental jadi saya pikir taruhan terbaik di sini adalah menggabungkan tslint dengan alat pengamat lain yang ada, seperti nodemon yang disebutkan di atas atau https: //www.npmjs.com/package/watch

membutuhkan perancangan ulang secara radikal untuk mendukung operasi ulang inkremental

FWIW kedengarannya sangat sulit, tetapi juga lebih berkinerja daripada menjalankan kembali semua tslint saat perubahan, seperti yang akan terjadi dengan alat pihak ketiga. +1 untuk permintaan ini, bersama dengan sisi pemahaman yang sehat.

ini.

Alat saya sebenarnya mendukung perilaku ini, karena saya mengerti bahwa ini sangat sulit untuk diterapkan. https://github.com/guidojo/multipleTypescriptCompilers#readme

Alat tersebut saat ini menjalankan kompiler tsc dan setelah setiap kali dijalankan, alat ini akan linting setiap file secara terpisah. Jika terganggu oleh kompilasi lain, itu akan dibatalkan. :)

Apakah ada yang melihat ke dalam ini belum?

Sementara itu, saya telah menggunakan paket onchange dalam skrip npm saya: https://github.com/alexburner/chain-of-being/blob/85230a5b0bf06e1e0a729559c340493c93dac008/package.json#L18

Setelah melihatnya di dokumen prettier : https://prettier.io/docs/en/watching-files.html

Menggunakan paket onchange berfungsi dengan baik, terima kasih banyak atas sarannya @alexburner. Saat ini menggunakannya untuk tslint dan stylelint:

{
    "lint": "concurrently \"yarn tslint:once\" \"yarn stylelint:once\" && onchange 'src/**/*.*' -- concurrently \"yarn tslint:once\" \"yarn stylelint:once\"",
    "tslint:once": "tslint -p . -c ./tslint.json --fix './src/**/*.+(ts|tsx)'",
    "stylelint:once": "stylelint --fix **/*.scss",
}

Saya pikir solusi onchange sudah cukup untuk kasus penggunaan CLI. Anda dapat menggunakan Node API dan mengimplementasikan pengamat di atasnya jika Anda mau. Saya akan menutup ini karena kami ingin memfokuskan upaya pengembangan di tempat lain di TSLint.

Saya tidak setuju bahwa ini harus ditutup, dan setidaknya harus tetap pada peta jalan bahkan jika fokus saat ini ada di tempat lain. Menonton tampaknya terlalu rumit dibandingkan dengan tsc -w . tsc

Apakah halaman ini membantu?
0 / 5 - 0 peringkat