Tslint: Peringatan Ditampilkan Untuk Aturan `no-unused-variable` yang Didukung

Dibuat pada 13 Sep 2017  ·  3Komentar  ·  Sumber: palantir/tslint

Laporan Bug

  • __TSLint versi__:
$ $(yarn bin)/tslint --version
5.7.0
  • __TypeScript versi__:
$ $(yarn bin)/tsc --version
Version 2.5.2
  • __Menjalankan TSLint melalui__: CLI

Perilaku sebenarnya

Saya menjalankan tslint atas basis kode yang ada.

Proyek saya memiliki set aturan no-unused-variable (melalui penggunaan paket seperti preset).

Proyek saya juga memiliki opsi kompiler noUnusedLocals dan noUnusedParameters diaktifkan untuk TypeScript (melawan, melalui penggunaan paket seperti preset).

Saya kadang-kadang memperbarui aturan saya dan menjalankan opsi tslint 's --fix untuk memperbaiki kode saya secara otomatis.

Itu termasuk memanfaatkan fitur perbaikan otomatis dari aturan no-unused-variable .

Contoh keluaran saat menjalankan tslint --fix pada basis kode saya:

$ $(yarn bin)/tslint --project tsconfig.json --config tslint.json --fix src/**/*.ts
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.
WARNING: 'no-unused-variable' lint rule does not need to be set if the 'no-unused-locals' and 'no-unused-parameters' compiler options are enabled.

Perilaku yang diharapkan

Seperti disebutkan di sini, aturan no-unused-variable tidak digunakan lagi karena pengguna (termasuk saya sendiri) mendapat manfaat dari fitur perbaikan otomatis.

Namun peringatan itu menyiratkan bahwa saya tidak boleh menggunakan aturan sama sekali, dan menyebabkan banyak keluaran yang tidak perlu ke konsol, yang dapat mengaburkan informasi penting lainnya seperti masalah yang tidak dapat diperbaiki dalam kode.

Saya percaya akan baik-baik saja menghapus peringatan itu.

P3 Accepting PRs Bug
Sumber
picture ghost

Komentar yang paling membantu

@destroyerofbuilds kedengarannya seperti poin yang bagus -- Anda menggunakan no-unused-variable terutama untuk fungsi perbaikan otomatis, jadi peringatannya tidak relevan. Saya pikir kita harus menghapus peringatan (menerima PR). Alangkah baiknya juga memperbarui dokumen aturan untuk menjelaskan pola penggunaan ini.

picture adidahiya pada 14 Sep 2017
👍2

Semua 3 komentar

Bagi saya, variabel no-unused-tidak melakukan apa-apa ;(
Versi 5.7.0

Ini adalah konfigurasi saya

{
    "extends": [
        "tslint:latest",
        "tslint-react",
        "tslint-config-prettier"
    ],
    "rules": {
        "ban-types": false,
        "interface-name": null,
        "interface-over-type-literal": false,
        "jsx-boolean-value": null,
        "jsx-no-lambda": null,
        "max-classes-per-file": [],
        "member-access": false,
        "no-namespace": null,
        "no-unused-expression": [false],
        "no-unused-variable": true,
        "no-var-requires": false,
        "object-literal-shorthand": false,
        "object-literal-sort-keys": false,
        "only-arrow-functions": null,
        "ordered-imports": null,
        "prefer-const": false,
        "variable-name": false,
        "no-invalid-template-strings": false,
        "no-submodule-imports": false
    }
}
tomitrescak picture tomitrescak pada 14 Sep 2017

@destroyerofbuilds kedengarannya seperti poin yang bagus -- Anda menggunakan no-unused-variable terutama untuk fungsi perbaikan otomatis, jadi peringatannya tidak relevan. Saya pikir kita harus menghapus peringatan (menerima PR). Alangkah baiknya juga memperbarui dokumen aturan untuk menjelaskan pola penggunaan ini.

adidahiya picture adidahiya pada 14 Sep 2017
👍2

@tomitrescak bagaimana Anda menjalankan TSLint? Kedengarannya seperti masalah terpisah dari posting asli; Saya akan mendorong Anda untuk membuka edisi lain jika Anda belum melakukannya.

adidahiya picture adidahiya pada 14 Sep 2017
Apakah halaman ini membantu?
0 / 5 - 0 peringkat

Masalah terkait

jamesarosen picture jamesarosen  ·  3Komentar
mrand01 picture mrand01  ·  3Komentar
Ne-Ne picture Ne-Ne  ·  3Komentar
ghost picture ghost  ·  3Komentar
denkomanceski picture denkomanceski  ·  3Komentar