Js-beautify: v1.8.0 게시

LGTM

나는 이것에 대해 매우 흥분된다. 서두를 생각은 없었지만, 출시 후보들을 기대하며 지켜봤다. 내가 #1033에 대한 수정 사항을 게시하고 싶어한다는 것을 알고 있습니다.

나는 atom-beautify를 최신 v0.33.0으로 업데이트한 다음 해당 수정 사항을 테스트하기 위해 #1033의 지침을 따르려고 했습니다. 내 노트에 따르면, 나는의 전체 내용을 복사 js/lib 에 JS-아름답게 유통에 ~/.atom/packages/atom-beautify/node_modules/js-beautify/js/lib 거기 원자 아름답게 파일을 대체합니다.

그러나 이제 더 이상 js/lib 디렉토리가 없는 것 같습니다. 최신 js-beautify-1.8.0-rc12.zip 도 아니고 master (3374af3)의 커밋 풀에도 없습니다. 일부 단계를 놓쳤거나 배포 레이아웃이 변경되었습니까?

@garretwilson

아 예, 죄송합니다. 파일은 더 이상 마스터에 체크인되지 않습니다.
웹 사이트가 제공되는 gh-pages 지점에서 사용할 수 있습니다.
https://github.com/beautify-web/js-beautify/tree/gh-pages/js/lib에 있지만 다운로드하려면 여러 단계가 필요합니다.

그러나 여기에 lib 디렉토리의 zip이 있습니다. 이에 대해 이전과 동일한 지침을 따르십시오.
lib.zip

저 역시 인라인 요소를 수정하게 되어 매우 기쁩니다. 또한 선택적 html 태그에 대한 기본 지원이 있다는 점도 기쁘게 생각합니다. 따라서 이 입력은 최신 버전에서 변경되지 않은 상태로 유지되지만 이전에는 과도하게 들여쓰기됩니다.

<ul>
    <li>Item 1
    <li>Item 2
    <li>Item 3
</ul>

또한 https://github.com/beautify-web/js-beautify/archive/gh-pages.zip 에서 얻을 수 있습니다.

파일은 더 이상 마스터에서 체크인되지 않습니다.

그래서 atom-beautify에서 어떻게 사용되는지 확실하지 않습니다. 저는 Atom 플러그인 시스템이 어떻게 작동하는지 전문가가 아닙니다. Atom은 자동으로 js-beautify 종속성을 빌드하고 Atom 플러그인 설치 절차의 일부로 이러한 파일을 생성합니까? 아니면 @Glavin001 이 js-beautify를 통합하기 위해 지금 추가 작업을 해야 합니까?

@garretwilson

최종 릴리스에서 atom 플러그인은 npm 패키지에서 파일을 가져옵니다. https://registry.npmjs.org/js-beautify/-/js-beautify-1.8.0-rc12.tgz

일반적인 사용에는 변경 사항이 없습니다. 우리는 master 브랜치에서 빌드된 파일이 흥미롭지 않고 어떤 파일을 편집해야 하는지 알아내려고 하는 기여자들에게 혼란을 야기했기 때문에 체크 인을 중단했습니다.

Brackets 확장의 경우 나머지 패키지가 필요하지 않기 때문에 일반적으로 해당 폴더에서 파일을 가져옵니다.
GitHub의 릴리스 에도 추가할 수 있습니까?

@bitwiseman에 대한 지속적인 노력에 감사드립니다!

1.8.0-rc12를 시도하는 동안 문제가 발생했습니다. 이에 대한 문제 를 제출했습니다.

@astronomersiva - 감사합니다! @MacKLess 가 수정된 PR을 제출한 것 같습니다.

@Hirse - npm에서 이미 사용 가능한 것을 다시 실행하지 않는 것이 좋습니다.
npm 패키지에서 파일을 쉽게 업데이트할 수 있는 https://github.com/brackets-beautify/brackets-beautify/pull/266 을 제출했습니다. 사용자 컴퓨터에서).

내 온전성 검사를 통과했습니다. 미화 출력이 좋아 보이고 여전히 정상입니다.
이 릴리스에 대해 매우 흥분됩니다. 도움을 주신 모든 기여자에게 수고했습니다.

그러나 여기에 lib 디렉토리의 zip이 있습니다.

나는 이것을 최신 atom-beautify v0.33.0(#1033을 확인하는 데 사용한 것과 동일한 버전에 가깝다)으로 테스트했는데 충돌했습니다.

ReferenceError: token is not defined
    at Beautifier.beautify (C:\Users\user\.atom\packages\atom-beautify\node_modules\js-beautify\js\lib\beautify-html.js:1530:29)
    at style_html (C:\Users\user\.atom\packages\atom-beautify\node_modules\js-beautify\js\lib\beautify-html.js:1150:21)
    at exports.html_beautify (C:\Users\user\.atom\packages\atom-beautify\node_modules\js-beautify\js\lib\beautify-html.js:2258:16)
    at file:///C:/Users/user/.atom/packages/atom-beautify/src/beautifiers/js-beautify.coffee:48:20
    at Promise._execute (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\debuggability.js:303:9)
    at Promise._resolveFromExecutor (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:483:18)
    at new Promise (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:79:10)
    at JSBeautify.module.exports.JSBeautify.beautify (file:///C:/Users/user/.atom/packages/atom-beautify/src/beautifiers/js-beautify.coffee:32:12)
    at file:///C:/Users/user/.atom/packages/atom-beautify/src/beautifiers/index.coffee:361:28
    at tryCatcher (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\util.js:16:23)
    at Promise._settlePromiseFromHandler (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:512:31)
    at Promise._settlePromise (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:569:18)
    at Promise._settlePromiseCtx (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\promise.js:606:10)
    at Async._drainQueue (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:138:12)
    at Async._drainQueues (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:143:10)
    at Async.drainQueues (C:\Users\user\.atom\packages\atom-beautify\node_modules\bluebird\js\release\async.js:17:14)
    at <anonymous>

(이것은 내가 시도한 4개의 파일 중 3개에서 발생했습니다.)

좋아 보이지 않습니다. 이것을 공개하지 마십시오.

@amandabot 축하 드립니다. 😄

@garretwilson
문제 없습니다. 시도해 주셔서 감사합니다. 내일 rc13이 있을 것입니다(#1496 및 수정 PR #1499 참조).

내일 rc13이 있을 것입니다(#1496 및 수정 PR #1499 참조).

PR #1499는 댓글을 보니 정말 걱정스럽습니다. 비분리 공간을 일반 공간으로 변경하여 내용을 변경하지 않는 해결 방법이 되길 진심으로 바랍니다. 콘텐츠를 손상시키지 마십시오. 일반적인 깨는 공백을 감싸고 나머지는 그대로 둡니다.

@garretwilson
rc13이 출시되었습니다.
https://github.com/beautify-web/js-beautify/archive/gh-pages.zip을 다시 다운로드하여 최신 파일을 받으세요.
@astronomersiva - 당신의 문제는 지금 수정되어야 합니다.

rc13에 관해서는 좋은 소식과 나쁜 소식이 있습니다. 첫 번째 좋은 소식은 더 이상 충돌하지 않는다는 것입니다. 만세!

더 좋은 소식 : 나는 인라인 요소가 마지막으로 올바르게 포맷되고 있다고 기뻐했다 방법 # 1033에 기억,하지만 난 언급 한 <figcaption> 내부 <figure> 가 따른다면, 줄 바꿈을 받고되지 않았습니다 <img/> 와 같은 인라인 요소:

    <figure class="near side"><img src="images/flyway-schema_version-table.png" alt="Flyway schema_version table." /> <figcaption>Flyway schema_version table. (<a href="https://flywaydb.org/getstarted/how"><cite>How Flyway works</cite></a>)</figcaption>
    </figure>

이것은 큰 문제가 아니었고 나는 그것을 가지고 살 수있었습니다. 그래도 저는 이것이 이제 rc13에서(그리고 충돌하지 않았을 때 rc12에서도) 수정되었다고 말하게 되어 기쁩니다!

    <figure class="near side"><img src="images/flyway-schema_version-table.png" alt="Flyway schema_version table." />
        <figcaption>Flyway schema_version table. (<a href="https://flywaydb.org/getstarted/how"><cite>How Flyway works</cite></a>)</figcaption>
    </figure>

또한 </figure> 앞에 줄 바꿈이 있는 </code></pre></figure> 와 같이 <figure> 내부의 다른 형식을 개선해야 합니다.

그러나 문제가 있습니다. 중첩 목록을 들여쓰기하는 데에는 상당히 큰 회귀가 있습니다. 알고리즘은 이제 목록 수준에 대해 혼란스러워 하고 중첩 목록의 목록 항목은 _backwards_ 들여쓰기됩니다! 문제 #1501을 제출했습니다.

(HTML과 관련이 없지만 CSS에서 @media 블록 내부의 규칙은 이제 세로로 강제로 묶이는 대신 빈 줄로 구분됩니다. 좋습니다. 감사합니다.)

@garretwilson 1.8.0-rc14 게시됨 - #1501 수정.

또한 선택적 html 태그에 대한 기본 지원이 있다는 사실을 기쁘게 생각합니다.

처음에 이걸 봤을 때 비꼬는 댓글을 달고 싶었습니다. 나는 사람들이 왜 형식이 좋지 않은 콘텐츠를 _원하는지_ 이해하지 못할 것입니다. 엔딩 태그 붙이는게 그렇게 어렵나요?

문제는 선택적 종료 태그가 있을 때 갑자기 콘텐츠를 구문 분석하기가 정말 어렵다는 것입니다(여기에서 해당 사례를 설명하는 확실하지 않은 명확한 문법을 ​​엄격하게 따르는 잘 설계된 파서가 없는 경우). 모호함을 해결하려면 그 사람이 의도한 바를 "추측"해야 하고, 사람들이 의도한 바를 추측하는 것이 사람들에게 어렵다면 컴퓨터에 대해 상상해 보십시오.

따라서 구문 분석기가 가장 잘 구성되지 않은 콘텐츠로 가장 게으른 작성자를 처리하도록 유연하게 하려면 괜찮습니다. 스펙에 맞게 잘 짜여진 콘텐츠를 만들기 위해 최선을 다하는 저희들의 콘텐츠가 깨지지 않도록 각별히 주의하시기 바랍니다.

나는 당신이 작은 비누 상자 연설을 용서하기를 바랍니다. 업계 전반에서 볼 수 있는 문제입니다.

오늘 #1501에 대한 수정 사항을 테스트하려고 하지만 지금은 전반적으로 콘텐츠에 대해 더 걱정하고 있습니다.

(PS 예, HTML이 이를 허용한다는 것을 알고 있습니다. 예, HTML이 XML과 호환되지 않는다는 것을 알고 있습니다. 저는 단지 일반적인 감정을 표현하는 것입니다.)

이 문제에 대한 수정 사항을 제출할 때 p와 관련된 새 문제도 제출했습니다.
특히 태그를 사용하는 사람들의 콘텐츠를 깨고 싶지 않았기 때문에
당신이 말했듯이, "우리는
사양." 한 번에 해결하기에는 가능한 버그가 너무 많다는 것을 알고 있었습니다.
앉았고, 솔직히 말해서, 그것에 대한 느낌이 없었습니다. 바라건대, 이 수정 사항은
덜 간단한 사람들에게 유연성을 제공하기에 충분히 간단합니다.
사람들에게 해를 끼치지 않으면서 닫는 태그로 엄격합니다.

2018년 8월 23일 목요일 오후 2:14 Garret Wilson 알림 @github.com
썼다:

또한 선택적 html 태그에 대한 기본 지원이 있다는 사실을 기쁘게 생각합니다.

처음에 이걸 봤을 때 비꼬는 댓글을 달고 싶었습니다. 나는 결코
사람들 이 형식이 좋지 않은 콘텐츠를 만들고
종료 태그를 추가하는 것이 정말 그렇게 어렵습니까?

문제는 선택적 종료 태그가 있을 때 갑자기
콘텐츠를 구문 분석하기 정말 어렵게 만듭니다(잘 설계된
명확하지 않은 문법을 엄격하게 따르는 파서
여기의 경우). 모호성을 해결하려면 무엇을 "추측"해야 합니다.
사람이 의도한 것이고, 사람이 의도한 것이 무엇인지 추측하기 어렵다면,
컴퓨터를 상상해보십시오.

따라서 파서가 가장 게으른 데이터를 처리할 수 있도록 유연하게 하려면
가장 형식이 좋지 않은 콘텐츠를 가진 작성자는 괜찮습니다. 매우 조심하십시오
우리의 콘텐츠를 시도하는 사람들의 콘텐츠를 손상시키지 않도록
사양에 맞게 잘 구성된 콘텐츠를 만드는 것이 가장 좋습니다.

나는 당신이 작은 비누 상자 연설을 용서하기를 바랍니다. 내가 볼 수있는 문제입니다
업계 전반에 걸쳐.

#1501에 대한 수정 사항을 테스트하려고 합니다.
https://github.com/beautify-web/js-beautify/issues/1501 오늘, 하지만 지금
나는 전반적으로 콘텐츠에 대해 더 걱정한다.

—
당신이 언급되었기 때문에 이것을 받는 것입니다.
이 이메일에 직접 답장하고 GitHub에서 확인하세요.
https://github.com/beautify-web/js-beautify/issues/1495#issuecomment-415573260 ,
또는 스레드 음소거
https://github.com/notifications/unsubscribe-auth/Acnku-jlxTrkFfhywQJ_Vrwt9yjNxjXZks5uTxtHgaJpZM4WHN3o
.

두 가지 유형의 작가 모두에게 효과가 있다면 좋습니다. 나는 단지 내가 생각하는 것들에 대한 접선을 벗어났습니다. 😁

@bitwiseman , atom-beautify용 rc14 파일에 편리한 Zip 파일이 있습니까?

@garretwilson https://github.com/beautify-web/js-beautify/archive/gh-pages.zip을 다시 다운로드하십시오.

나는 오늘 이것을 얻지 못할 것 같다. 일이 바빠서 몇 분 쉬어야 해요.

하지만 이것에 대해 더 생각해보면, 나는 여전히 닫히지 않은 요소를 처리하는 이 새로운 기능에 대해 약간 걱정하고 있습니다. 코너 케이스에서 올바르게 작동하고 기존 콘텐츠에 해를 끼치지 않는지 확인하기 위해 많은 심층 테스트 케이스가 코드에 추가되었습니까? 14번째 릴리스 후보에 이런 버그가 떠오를까봐 조금 걱정입니다. 내 걱정이 불필요하고 단위 테스트 등이 많다면 내가 그것을 꺼내는 것을 용서하십시오. 나는 그 코드를 보지 않았다. 여러분은 그것이 얼마나 안전하고 테스트되었는지 더 잘 다룰 것입니다.

나는 계속해서 오늘 밤 이것을 테스트하는 시간을 가졌습니다.

이 rc14는 들여쓰기가 제거된 중첩 정의 목록 <dl> / <dt> / <dd> 에서 여전히 버그를 표시합니다. 티켓 #1507을 제출했습니다.

14번째 릴리스 후보에 코드베이스의 버그가 너무 많이 나타나면 코드가 프로덕션 준비가 되지 않은 것이 분명합니다.

릴리스 스트림에 넣기 전에 이 새 코드를 제거하고 충분한 단위 테스트를 제공하도록 요청합니다. 해당 코드 없이 릴리스 후보를 생성하고 코드 없이 1.8.0을 릴리스하기를 바랍니다.

나는 문자 그대로 #1033이 수정되기를 몇 년 동안 기다렸고 조언과 돈을 기부한 후 문제가 해결된 것을 보고 기뻤습니다. 나는 당신이 그 수정 사항이 포함된 버전을 출시하기를 바랐지만 지금은 모든 것이 지연되고 v1.8.0을 기다리며 새로운 것을 추가하는 것처럼 보입니다. #1033에 대한 수정 사항을 야생으로 가져올 수 없습니까?

이제 문제는 #1033 수정으로 돌아가려면 #1033이 수정된 지점으로 돌아가서 파일을 복사해야 일상적인 작업을 할 수 있다는 것입니다. 일하다.

모든 불만에 대해 죄송합니다. #1033이 "아직 출시됐어? 아직 출시됐어?" 매일 질문.

알겠습니다. 꽤 피곤합니다. 내일 상황을 확인하겠습니다. 안녕히 주무세요.

#1507에서 언급했듯이 r14 대신 테스트할 때 r13에 지쳐서 압축을 풀었습니다. 따라서 들여쓰기가 제거된 중첩 정의 목록은 r14에 나타나지 않습니다. 내 실수. 원래 의도한 대로 테스트하려면 내일까지 기다려야 했습니다. 하지만 발매가 되어서 너무 걱정이 됩니다!

어쨌든 전반적인 우려 사항은 남아 있지만 언급했듯이 새 코드가 얼마나 견고한지 여러분이 저보다 더 잘 알고 있습니다.

@garretwilson
이것은 모든 소프트웨어 릴리스에서 발생하며 이 경우 프로세스의 일부가 되고 보고 있습니다. 😄

당신이 어떻게 느끼는지 이해하지만 이 문제를 연 이후로 두 가지 버그만 추가되었습니다. 하나는 유니코드 및 공백 처리의 까다로운 영역에 있었고 다른 하나는 선택적 닫기 태그의 새로운 기능에 관한 것이었습니다. 둘 다 쉽게 이해되고 수정되었습니다. 문제는 체계적이지 않고 구체적입니다.

거의 다 왔어. 하루나 이틀만 더 rc14를 두드리면 문제가 해결될 것입니다.

알겠습니다. 여기가 조용해졌습니다. 좋아요.

나는 금요일에 이와 같은 거대한 릴리스를 발송하지 않을 것입니다.

그러나 큰 것이 들어오지 않는 한 1.8.0은 월요일에 일찍 나올 것입니다.

훌륭한 일!

방금 보고됨: #1514 및 https://github.com/beautify-web/js-beautify/issues/781#issuecomment -416342735

감사 해요!

@gabrielmaldi 그리고 #1514 _before_ 출시에 대해 말해주지 않으셨나요? 😄
릴리스 1.8.1.

1.8.1 파일을 받을 수 있는 Zip이 있습니까? @Glavin001 이 이것을 atom-beautify에 통합하기를 기다리고 있으므로 수동으로 설치 파일을 계속 업데이트해야 합니다. 감사 해요.

@Glavin001 이 이것을 atom-beautify에 통합하기를 기다리고 있으므로 수동으로 설치 파일을 계속 업데이트해야 합니다.

@garretwilson : js-beautify 대한 Atom-Beautify의 버전 범위는 ^1.7.5 . https://github.com/Glavin001/atom-beautify/blob/master/package.json#L194

새 릴리스가 1.8.1이므로 Atom-Beautify를 제거하고 다시 설치하여 js-beautify 와 같은 이러한 종속성의 업데이트를 트리거할 수 있습니다. semver 일치에 대해서는 https://semver.npmjs.com/ 을 참조

위의 것이 불가능하더라도 js-beautify 종속성을 1.8.1 로 업데이트하기 위한 공개 Pull Request가 표시되지 않습니다. https://github.com/Glavin001/atom-beautify/pulls

js-beautify 또는 기타 종속성에 대한 변경 사항을 적극적으로 관찰하고 있지 않습니다 . 현재 우선순위는 https://unibeautify.com/ 인 Atom-Beautify의 미래입니다. 당신은 아톰 아름답게에 표시 할 업데이트가있는 경우, 둘 다 @ szeck87 및 I (@ szeck87 훨씬 더 내가 Unibeautify의 우선 순위를 봤는데으로, 최근 아톰 아름답게으로 참여하고있다)에 당신이 끌어 오기 요청을 제출하고 언급하는 것이 좋습니다 검토하고 병합하십시오. 감사 해요.

@Glavin001 ,

js-beautify에 대한 Atom-Beautify의 버전 범위는 ^1.7.5입니다. … 새 릴리스가 1.8.1이므로 Atom-Beautify를 제거하고 다시 설치하여 js-beautify와 같은 이러한 종속성의 업데이트를 트리거할 수 있습니다.

오, 멋지네요! 엄청난.

그러나 문제는 atom-beautify가 _duplicate_ 기본값을 하드 코딩하고(과거에 주의 했음) js-beautify #1033에 대한 수정을 위해 unformatted 기본값을 변경해야 하기 때문에 atom -beautify도 기본값을 변경해야 합니다.

https://github.com/Glavin001/atom-beautify/issues/2210 에서 요청했습니다. (만약 당신이 js-beautify의 기본 설정을 위임하여 중복된 기본 설정을 _제거_한다면, 우리는 js-beautify가 기본값을 조정할 때마다 이것을 계속할 필요가 없을 것입니다. 그것이 제가 선호하는 것입니다.)

나는 https://github.com/Glavin001/atom-beautify/issues/2210#issuecomment -417824930 에 댓글을 달았습니다. 거기에서 Atom-Beautify 관련 토론을 계속하십시오. 감사 해요!