Openlibrary: Mari kita lint JavaScript kita dengan eslint!

Dibuat pada 17 Okt 2018 · 5Komentar · Sumber: internetarchive/openlibrary

Stylelint memberi kita perlindungan dari berbagai masalah CSS. Mari kita lakukan hal yang sama untuk JS kita.
eslint adalah yang paling matang dan menyediakan daftar aturan yang dapat kita gunakan jika kita anggap perlu.

npm install eslint --save-dev

Di package.json, tambahkan perintah baru

"lint:js": "eslint openlibrary/plugins/openlibrary/js/"

Tambahkan .eslintrc.json

{
    "root": true,
    "env": {},
    "globals": {
    },
    "rules": {
        "no-extra-semi": "warn"
    }
}

Kami akan mengatur kesalahan untuk diperingatkan untuk saat ini dan segera setelah konfigurasi digabungkan, kami harus berkoordinasi dengan seseorang untuk menjalankan perbaikan dan menggabungkan untuk menghindari konflik penggabungan!

[ ] npm run lint:js akan menunjukkan kepada saya peringatan yang berkaitan dengan aturan "no-extra-semi".
[ ] "npm run lint:js " terjadi saat saya menjalankan "npm run test"
[ ] Jangan perbaiki kesalahan apa pun sebagai bagian dari perubahan ini!

Good First Issue Linting JavaScript 1 hacktoberfest

Sumber

jdlrobson

Semua 5 komentar

Hai. Bisakah Anda menugaskan saya untuk menyelesaikan masalah ini?

0x15F9 pada 18 Okt 2018

Pertanyaan: mengapa tidak menggunakan salah satu preset untuk eslint ? Seperti https://standardjs.com atau https://github.com/airbnb/javascript

tamtamchik pada 20 Okt 2018

Jangka panjang saya ingin tetapi javascript dalam kondisi yang buruk menerapkan preset seperti itu akan banyak membingungkan (review kode, konflik rebase) dan pekerjaan manual (sebagian besar preset tidak dapat diperbaiki secara otomatis). Saya pikir menambahkan aturan secara manual membantu kita menyerangnya sedikit demi sedikit dengan cara yang kohesif dan mendidik (kita belajar tentang masalah dalam kode kita sedikit demi sedikit).

Saya pikir kita bisa menggunakan linting untuk mengatasi masalah yang paling umum dan mendesak (serat yang dapat menyebabkan kesalahan dalam kode) dan kemudian di waktu luang kami menerapkan preset (pilihan kami) nanti di jalan.

Apakah ini masuk akal?

jdlrobson pada 20 Okt 2018

👍1

Tentu saja. Bentuk kode bisa sangat buruk.

Ketika PR ini akan digabungkan, saya dapat memperbaiki beberapa masalah dengan linting, sebutkan saja.
Akan mulai dengan no-extra-semi misalnya ;)

tamtamchik pada 20 Okt 2018

👍1

@tamtamchik saya dapat membuat masalah nanti tetapi apa pun yang ada di bawah "kemungkinan Kesalahan" snd dan dapat diperbaiki secara otomatis adalah permainan yang adil.

Kita harus berkoordinasi dengan @mekarpeles untuk menjalankan dan melakukan perbaikan otomatis dan membuat kesalahan eslint daripada memperingatkan.

Saya ingin kita menyingkirkan buah yang menggantung rendah!

jdlrobson pada 21 Okt 2018

🎉1

Apakah halaman ini membantu?

0 / 5 - 0 peringkat