Openlibrary: eslint로 자바스크립트를 lint하자!

에 만든 2018년 10월 17일 · 5코멘트 · 출처: internetarchive/openlibrary

Stylelint는 다양한 CSS 문제로부터 우리를 보호합니다. JS에 대해서도 동일한 작업을 수행해 보겠습니다.
eslint는 가장 성숙하고 필요하다고 판단되는 경우 사용할 수 있는 규칙 목록을 제공

npm install eslint --save-dev

package.json에서 새 명령을 추가합니다.

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

.eslintrc.json 추가

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

당분간 경고하도록 오류를 설정하고 구성이 병합되는 즉시 누군가와 협력하여 수정 사항을 실행하고 병합 충돌을 피하기 위해 병합해야 합니다!

[ ] npm run lint:js 는 "no-extra-semi" 규칙과 관련된 경고를 표시합니다.
[ ] "npm run lint:js "는 "npm run test"를 실행할 때 발생합니다.
[ ] 이 변경의 일부로 오류를 수정하지 마십시오!

Good First Issue Linting JavaScript 1 hacktoberfest

출처

jdlrobson

모든 5 댓글

안녕. 이 문제를 해결하도록 저를 지정할 수 있습니까?

0x15F9 에 2018년 10월 18일

질문: eslint 대한 사전 설정 중 하나를 사용하지 않는 이유는 무엇입니까? https://standardjs.com 또는 https://github.com/airbnb/javascript 와 같이

tamtamchik 에 2018년 10월 20일

장기적으로 원하지만 이러한 사전 설정을 적용하면 자바스크립트가 매우 좋지 않은 상태에 있으므로 많은 혼란(코드 검토, 리베이스 충돌)과 수동 작업(대부분의 사전 설정은 자동 고칠 수 없음)이 될 것입니다. 규칙을 수동으로 추가하면 응집력 있고 교육적인 방식으로 조금씩 공격하는 데 도움이 된다고 생각합니다(코드의 문제에 대해 조금씩 알아갑니다).

린트를 사용하여 가장 일반적이고 시급한 문제(코드에서 오류를 일으킬 수 있는 린트)를 해결한 다음 나중에 여가 시간에 사전 설정(선택한)을 나중에 적용할 수 있다고 생각했습니다.

이게 말이 돼?

jdlrobson 에 2018년 10월 20일

👍1

물론입니다. 코드 형태는 매우 나쁠 수 있습니다.

이 PR이 병합되면 Linting과 관련된 몇 가지 문제를 해결할 수 있습니다. 이름을 em으로 지정하면 됩니다.
예를 들어 no-extra-semi 로 시작합니다 ;)

tamtamchik 에 2018년 10월 20일

👍1

@tamtamchik 나중에 문제를 만들 수 있지만 "가능한 오류"에 있고 자동 수정 가능한 모든 것은 공정한 게임입니다.

@mekarpeles 와 협력하여

나는 우리가 낮은 매달린 과일을 제거하기를 원합니다!

jdlrobson 에 2018년 10월 21일

🎉1

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급

Openlibrary: eslint로 자바스크립트를 lint하자!

모든 5 댓글

관련 문제