Jshint: 탭 들여쓰기가 여러 규칙을 위반함

신고 감사합니다! 문제 는 lex.js 의 1610번째 줄 에서 비롯됩니다.

this.input = this.input.replace(/\t/g, state.tab);

이것은 얼마나 많은 스타일 관련 린트 규칙이 구현되는지에 있어 매우 기본적인 것 같습니다. 그것들은 이제 한동안 더 이상 사용되지 않지만 우리는 여전히 새로운 주요 버전으로 이동할 준비가 되지 않았습니다. 따라서 그동안 우리는 해당 동작을 보존하는 솔루션을 설계해야 합니다.

내 초기 생각은 이 "유효 오프셋"을 column 라는 토큰 개체의 새 속성으로 추적하는 것입니다. 이는 스타일 관련 경고에 사용할 수 있습니다. 그런 다음 character 속성을 다시 구현하여 "true" 문자 오프셋을 설명할 수 있습니다. 여기서 각 코드 포인트(탭 또는 기타)는 카운트를 정확히 1만큼 증가시킵니다. 이 값은 경고를 발행하는 데 사용됩니다. 리포터 플러그인은 탭 문자를 렌더링하는 방법을 제어하고 보고된 "열" 번호를 그에 따라 해석할 수 있습니다.

@Arcanemagus, 잘 들리나요?

적어도 JSHint의 내부에 대한 실제 지식이 없는 상태에서 표면적으로는 완벽하게 들립니다 😛. 이상적으로는 실제 열 수(각 코드 포인트가 하나의 열로 계산되는)를 얻는 몇 가지 방법이 필요합니다.

이것은 반 공통 오류가 있는 오래된 파서 버그에 대한 해결 방법으로 인해 본질적으로 무시되는 이 경고로 인해 잠시 숨겨졌습니다. 훨씬 좋은 방식으로 사용자에게 경고를 다시 구현했을 때 실수로 사용자를 _숨겼습니다_. 그리고 최근에 이를 다시 표시하도록 수정했습니다.

W030 , W033 및 W009 도 영향을 받는 것 같습니다. 귀하의 설명에서 _모든_ 규칙이 영향을 받는다고 가정합니까?

저도 이 문제를 겪고 있으며 오류와 경고를 모두 복제할 수 있습니다. 규칙/오류 유형은 중요하지 않은 것 같습니다.

다음은 내 '=='가 '==='여야 한다고 경고하려는 jshint의 예입니다. 주황색 밑줄의 잘못된 위치를 확인하십시오.

이 문제에 대한 귀하의 생각이 확인되기를 바랍니다.

오늘은 파헤칠 시간이 있습니다. indent 옵션을 1 로 설정하면 원하는 결과를 얻을 수 있을 것입니다.

@Arcanemagus 소비자를 위한 indent 옵션의 가치를 무시할 수 있습니까? 이것이 가능하다면 JSHint 자체의 기본값을 변경하는 것이 소비자에게 어떤 영향을 미칠 수 있는지 잘 모르기 때문에 차라리 그렇게 하고 싶습니다.

흠, 현재 이 프로젝트는 CLI 인터페이스를 사용하고 있습니다. 이를 가능하게 하는 Node.js API를 사용하도록 다시 작성할 수 있습니다.

• indent 값을 강제로 적용하면 소비자에게 표시되는 결과가 변경됩니까? 해당 패키지의 목표는 jshint 자체를 실행할 수 있는 한 가능한 한 가깝지만 결과를 편집기에 통합하는 것입니다.
• JSHint API에 파일에 대한 구성을 가져오는 숨겨진 메소드가 있습니까? 아니면 구현해야 합니까? 현재 문서 에는 그러한 기능이 표시되어 있지 않습니다.

구성에서 값을 재정의할 수 있는 명령줄 인수를 추가하는 것은 어떻습니까?
그런 것은 nginx의 -g, ssh의 -o, git의 -c 등과 같은 도구에서 일반적입니다.

$ jshint -o "indent = 1" -o "-W034 = true" -o "globals.require = false" -o "globals.$ = null"

이거 어떤 버전에 도입되었는지 아시는 분 계신가요? 2.9.5?

적어도 잠시 동안 임시로 다운 그레이드 할 수 있습니다. 고치다.

@jaredatch https://github.com/AtomLinter/linter-jshint/pull/386(v3.1.0 of linter-jshint 릴리스)에서 JSHint의 버그가 있는 지점 보고에 대한 모든 해결 방법이 제거되었습니다. 그 검사의 요점은 린터가 잘못된 점을 보고하는 것과 같은 버그를 찾는 것이므로 린터를 사용하는 _모든 사람_에 대해 보고되고 수정될 수 있습니다. (결국 정확한 데이터를 제공해주는 린터를 믿을 수 없다면 왜 사용하는 걸까요?)

이 탭 버그가 도입되기 전과 이러한 해결 방법을 처음에 배치하게 만든 주요 파서 버그 이후에 "스위트 스팟"이 _may_ 있습니다.

@Arcanemagus gotcha , 통찰력에 정말 감사드립니다. 멋진 작업을 계속하십시오 👍

linter-jshint 에서 이러한 잘못된 포인트를 보고하는 방법에 대한 더 나은/더 유용한 메시지 아이디어가 있다면 자유롭게 문제를 제기하거나 거기에 PR을 제출하세요 😉.

빌어먹을 자바스크립트 파일을 2017년에 린트하려고 지난 5시간 동안 당신의 모든 솔루션을 제대로 사용하지 못했습니다.

그리고 도대체 탭을 사용하여 문자 그대로 들여쓰기를 하지 않는 이유가 무엇입니까?

누가 이 깨진 영어를 영어라는 수백만 명에게 공개하는 이유와 함께 문자 그대로 마침표를 사용하지 않습니까?

오. 그리고 대문자는 문장의 시작 부분에 옵니다. 모든 곳이 아닙니다.

/트롤 (미안해, 어쩔 수 없었어)

글쎄, 솔루션이 당분간 이것을 비활성화 할 것이라고 생각합니다 ...
어쨌든 누가 린터를 필요로 합니까?

@jugglinmike https://github.com/jshint/jshint/issues/3151#issuecomment -312512856의 질문에 답할 수 있습니까?

이 트리거되는 다양한 경우에 대해 20개의 고유한 문제가 열려 있습니다. 여기에서 이 문제를 수정하는 데 시간이 걸릴 것 같기 때문에 사용자의 결과를 변경하지 않을 것이라고 가정하고 해결 방법을 계속 진행하고 싶습니다.

이 문제는 내 레이더에서 벗어났습니다. 죄송합니다, @Arcanemagus. 불행히도, 나는 이번 주말까지 마땅히 받아야 할 관심을 줄 수 있을 것이라고 생각하지 않습니다. 그게 당신을 위해 작동합니까?

@jugglinmike 물론입니다 😉

기존 문제에 대해 몇 가지 확인을 했기 때문에 대부분의 사람들은 여기를 가리키는 이미 열린 문제로 필터링됩니다.

@cobexer 저는 그 아이디어가 마음에 듭니다. 왜냐하면 우리에게 앞으로 나아갈 길을 제시할 뿐만 아니라
다른 상황에 있는 사람들에게 유용할 수 있습니다. 즉, 내가 아는 한 거기에서
그런 종류의 기능을 요청한 적이 없으며
새 기능이 보기에 좋다는 이유만으로 커밋합니다. 그리고 그럴 이유가 있다
특히 해당 기능을 권장하지 않습니다. "rc" 파일 사이, 인라인
구성 및 Node.js API를 사용하면 사용자가 원할 때 이미 걸려 넘어집니다.
주어진 옵션이 활성화된 이유를 이해합니다. 구성을 위한 다른 벡터 추가
경영진은 이 문제를 악화시키는 경향이 있습니다.

여기에서 가장 실행 가능한 솔루션은 이전제안 .
그러나 우리는 character 속성을 공개 API인 Atom으로 간주해야 합니다.
결국 Editor 플러그인 자체가 빌드됩니다. 그래서 그것을 수정하는 대신
현재의 의미를 가진 새로운 속성을 정의하고,
character 를 그대로 두고 요청된 데이터를 새
속성 이름.

이번 주말에 새로운 기능에 대한 작업을 시작했습니다. 나는 회귀를 피하고 싶다
모든 비용이 발생하므로 일관되게 주장하기 위해 프로젝트의 테스트를 확장해야 합니다.
예상 문자 번호. 이 일을 시작하면서 나는
먼저 해결해야 하는 테스트 스위트. gh-3174는
그 목표.

이 모든 것은 시간이 걸릴 것 같습니다. @Arcanemagus 내가 아는
당신은 이 버그의 직격탄을 맞고 있으며, 이 버그를
개선된 JSHint를 볼 수 있는 기회. 나는 또한 당신이 관심을 갖는 것을 좋아합니다
사용자에게 투명하지 않은 솔루션을 피합니다. 짧은 시간에
기간, 사용자에게 indent 옵션을 1 로 설정하도록 권장한 적이 있습니까?
이상적이지는 않지만 그렇게 하면 단기적인 문제를 해결해야 합니다.
이것은 우리가 이것을 패치한 후에도 계속 작동할 것입니다.

@Arcanemagus 당신이 이 버그의 직격탄을 맞고 있다는 것을 알고 있으며, 이것을
개선된 JSHint를 볼 수 있는 기회.

대부분의 Atom용 Linter 제공자가 사용하는 일반 라이브러리에 검사를 추가한 모든 이유는 이와 같은 버그가 소스 린터에서 포착되어 보고/수정될 수 있기 때문입니다 😉. 이것은 실제로 더 일찍 포착되었을 것이지만 파서가 대부분의 문서에서 완전히 실패하고 보고할 좋은 방법이 없었을 때 기본적으로 _way_에서 모든 잘못된 포인트 오류를 ​​숨겼던 linter-jshint 에 확인이 있었습니다. 사용자 또는 중복 제거 문제 보고서. 이제 그것들이 파악되었으므로 해당 수표를 제거했고 여기 😛가 있습니다.

또한 사용자에게 투명하지 않은 솔루션을 피하는 데 관심이 있다는 점도 마음에 듭니다.

사실, 이 특정 문제에 대한 투명한 솔루션은 괜찮을 것입니다. indent 설정을 1 로 강제하면 그들이 얻을 수 있는 것과 비교하여 그들이 보고 있는 결과가 바뀔지 모르겠습니다. jshint 자체를 실행합니다.

단기적으로 사용자에게 들여쓰기 옵션을 1로 설정하도록 권장한 적이 있습니까?

그건... 사실 그 문제에 글을 올릴 생각을 하지 않았다는 훌륭한 아이디어입니다. 나는 문제에 대한 더 많은 의견에서 그것을 언급하도록 할 것입니다!

예, .jshintrc에서 'indent' 속성을 1로 설정하는 것은 훌륭한 해결 방법입니다. 나는 그것에 대해 더 빨리 알았더라면 좋았을 텐데. jshint가 기본적으로 es6으로 설정되어 있지 않기 때문에 이미 .jshintrc가 있으므로 추가하기 쉽습니다. 👍

반갑습니다, @tustin2121!

업데이트: 저는 여전히 회귀를 피하기 위해 JSHint 테스트 스위트를 강화하기 위해 노력하고 있습니다. 그 노력의 최신작은 gh-3176입니다.

이 문제에 대한 업데이트가 있습니까?

현재 이 버그의 영향을 받는 규칙이 최소 40개 이상인 것 같습니다 😛.

이것에 대한 모든 업데이트,이 오류가 발생한 지 몇 달이 지났습니다.

안녕하세요 @xcrap입니다! 방금 댓글 스레드를 확인했는데 아무도 업데이트를 게시하지 않은 것 같습니다. 좋은 소식은 JSHint가 오픈 소스 프로젝트이므로 스트레스를 유발하는 경우 문제를 해결하는 데 도움을 줄 수 있다는 것입니다! 손을 빌려주시겠습니까? 기꺼이 조언해 드리겠습니다.

@jugglinmike 좋겠다! 나는 항상 작은 프로젝트에서 시도하지만 내 js 기술은 매우 기본적이고 제한적입니다. :)

이 버그에 50달러의 현상금을 걸었습니다. 다른 사람들은 이 링크를 통해 현상금을 늘리는 데 기여할 수 있습니다. https://www.bountysource.com/issues/46533252-tab-indentation-breaks-multiple-rules

@tzvipm 의 포크를 로컬로 가져오고 jshint를 다시 빌드하면 다른 사람이 이 문제에 걸려 넘어지는 경우 문제가 해결되었습니다.

git clone https://github.com/tzvipm/jshint
git remote add upstream [email protected]:jshint/jshint.git
git fetch upstream
git stash
git merge upstream/master
git mergetool --tool=opendiff
npm run build