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-beautify-1.8.0-rc12.zipも、 master （3374af3）からのコミットプルにも、 js/libディレクトリはもう存在しないようです。 いくつかのステップを逃しましたか、または配布レイアウトが変更されましたか？

@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 ：

ファイルはマスターにチェックインされなくなりました。

そのため、atom-beautifyでどのように使用されるのかわかりません。 私はAtomプラグインシステムがどのように機能するかについての専門家ではありません。 Atomは自動的にjs-beautify依存関係を構築し、Atomプラグインのインストール手順の一部としてこれらのファイルを生成しますか？ それとも、 @ Glavin001はjs-

@garretwilson

最終リリースでは、atomプラグインはnpmパッケージからファイルを取得します： https ：

通常の使用法に変更はありません。 マスターブランチでビルドされたファイルのチェックインを停止したのは、それらが面白くなく、編集する必要のあるファイルを見つけようとする寄稿者に混乱を引き起こしていたためです。

Brackets拡張機能の場合、パッケージの残りの部分は必要ないため、通常はそのフォルダーからファイルを取得します。
それらをGitHubのリリースに追加することもできますか？

この@bitwisemanにご尽力いただき、誠にありがとうございます。

1.8.0-rc12を試しているときに問題が発生しました。 この問題を提出しました。

@ astronomersiva-ありがとう！ @MacKLessが修正付きのPRを送信したよう

@ Hirse -npmからすでに利用可能なものをやり直し
https://github.com/brackets-beautify/brackets-beautify/pull/266を送信しました。これにより、npmパッケージからファイルを簡単に更新できます（ただし、インストールされないようにdevDependencyとしてjs-beautifyもあります）ユーザーマシン上）。

私の健全性チェックに合格しました。美容師の出力は見栄えがよく、私はまだ確かに正気です。
これがリリースされることにとても興奮しています。 助けてくれたすべての貢献者によくやった。

しかし、ここにlibディレクトリのzipがあります。

これを最新のatom-beautifyv0.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>内の他のフォーマットも改善されます。

しかし問題があります。 ネストされたリストをインデントすることにはかなり大きな回帰があります。 アルゴリズムはリストレベルについて混乱しているように見え、ネストされたリストのリストアイテムは_後方_にインデントされます！ 問題＃1501を提出しました。

（HTMLとは関係なく、CSSでは、 @mediaブロック内のルールが、垂直方向に強制されるのではなく、空白行で区切られていることがわかります。ありがとうございます。）

@garretwilson 1.8.0-rc14が公開されました-修正＃1501。

また、オプションのhtmlタグの基本的なサポートがあることを嬉しく思います。

これを最初に見たとき、私は簡単なコメントをするつもりでした。 なぜ人々が整形式でないコンテンツを作りたいのか理解できません。 終了タグを追加するのは本当に難しいですか？

問題は、オプションの終了タグがある場合、コンテンツの解析が突然非常に困難になることです（明確な文法に厳密に従った適切に設計されたパーサーがない限り、ここでケースを説明するかどうかはわかりません）。 あいまいさを解決するには、その人が何を意図しているかを「推測」する必要があります。人々が何を意図しているのかを推測するのが難しい場合は、コンピューターを想像してください。

したがって、パーサーを柔軟にして、最も怠惰な作成者を最も整形式でないコンテンツで処理できるようにしたい場合は、問題ありません。 仕様に合った整形式のコンテンツを作るために最善を尽くしている私たちのコンテンツを壊さないように十分注意してください。

私はあなたが小さな石鹸箱のスピーチを許してくれることを願っています。 それは私が業界全体で見ている問題です。

今日は＃1501の修正をテストしようとしますが、今では全体的なコンテンツについてもっと心配しています。

（PSはい、HTMLでこれが可能であることを知っています。また、HTMLがXMLに準拠していなかったことも知っています。私は、一般的な感情を表現しているだけです。）

この問題の修正を提出したときに、pに関連する新しい問題も提出しました。
タグを付けるのは、特に、次のような人々のコンテンツを壊したくなかったからです。
あなたが言うように、「に合わせた整形式のコンテンツを作るために最善を尽くしています
スペック。」これには、1つで取り組むにはあまりにも多くの可能性のあるバグがあることを私は知っていました。
座って、率直に言って、それに気が進まなかった。 うまくいけば、この修正は
少ない人々に柔軟性を提供するのに十分簡単
ある人を傷つけることなく、タグを閉じることで厳密に。

2：14 PMギャレットウィルソンの木、2018年8月23日には[email protected]
書きました：

また、オプションの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ファイルはありますか？

@garretwilsonhttps ：//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が修正されて以来、「もうリリースされていますか？もうリリースされていますか？」と煩わされないように、私がどのように舌を噛んでいるのかわかりません。 毎日質問します。

OK、私はかなり疲れています。 明日は状況を確認します。 おやすみなさい。

＃1507で述べたように、テスト時にr14の代わりにr13が疲れて解凍されたことが判明したため、インデント解除されたネストされた定義リストはr14に表示されません。 私の間違い。 当初の意図どおりにこれをテストするには、明日まで待つ必要がありました。 でもリリースされるのがとても心配です！

とにかく、私の全体的な懸念は残っていますが、私が述べたように、皆さんは私よりも新しいコードがどれほど堅実であるかをよく知っています。

@garretwilson
これはどのソフトウェアリリースでも発生します。この場合、プロセスを確認してその一部になります。 😄

私はあなたの気持ちを理解していますが、この号を開いてから追加されたバグは2つだけです。 1つは、Unicodeと空白の処理という難しい領域にあり、もう1つは、オプションの終了タグの新機能に関するものでした。 それらは簡単に理解され、修正されました。 問題は特定のものであり、体系的なものではありません。

もうすぐ着きます。 rc14を1日か2日叩くだけで、ドアの外に出ます。

さて、ここは静かになりました。 それは良い。

私は金曜日にこのような巨大なリリースを出荷するつもりはありません。

しかし、何か大きなものが入らない限り、1.8.0は月曜日の早い時間に出ます。

すごい仕事！

報告されたばかり：＃1514およびhttps://github.com/beautify-web/js-beautify/issues/781#issuecomment -416342735

ありがとう！

@gabrielmaldiそして、リリース前の＃1514について教えてくれませんでしたか？ 😄
1.8.1をリリースしました。

1.8.1ファイルから取得できるZipはありますか？ @ Glavin001がこれをatom-

@ Glavin001がこれをatom-

@garretwilson ：Atom- js-beautifyのバージョン範囲は^1.7.5です： https ：

新しいリリースは1.8.1なので、Atom-Beautifyをアンインストールして再インストールするだけで、 js-beautifyなどのこれらの依存関係の更新をトリガーできます。 semverのマッチングについては、https：//semver.npmjs.com/を参照してください。

上記が不可能だったとしても、 js-beautify依存関係を1.8.1に更新するためのオープンプルリクエストが表示されません： https ：

js-beautifyやその他の依存関係の変更を積極的に監視していません。 私の現在の優先事項は、 https： //unibeautify.com/であるAtom-Beautifyの将来です。 Atom-Beautifyで見たい更新がある場合は、プルリクエストを送信して、 @ szeck87と私の両方に言及することをお勧めします（ @ szeck87は、最近、Unibeautifyを優先しているため、Atom-

@ Glavin001 、

Atom-Beautifyのjs-beautifyのバージョン範囲は^ 1.7.5です。…新しいリリースは1.8.1なので、Atom-Beautifyをアンインストールして再インストールするだけで、js-beautifyなどのこれらの依存関係の更新をトリガーできます。

良いですね！ 素晴らしい。

しかし、問題は、atom-beautifyが_duplicate_ defaultsをハードコーディングしていることです（これは過去に警告しました）。js-beautify＃1033の修正では、デフォルト値のunformatted変更する必要があるためです。 -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関連の議論を続けてください。 ありがとう！