Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

akashic export html --minify --atsumaruを実行した際にバンドルされるスクリプトにminifyを適用 #1047

Merged
merged 1 commit into from Mar 9, 2022

Conversation

z0ero
Copy link
Contributor

@z0ero z0ero commented Mar 5, 2022

概要

akashic export html --minify --atsumaru
上記のようなコマンドオプションを指定した際に生成されるzip内のfilesフォルダ以下に配置されるスクリプトがminifyされていないのを修正。

目的

ファイルサイズの縮小とニコ生でプレイする際のパフォーマンス改善に役立ちます。

@xnv
Copy link
Member

xnv commented Mar 8, 2022

ありがとうございます。 --atsumaru はニコ生ゲーム向けに適したオプションを勝手に使うオプションなので、 stiripbabel などと同じように常に true としてしまうのが良さそうです。

@z0ero
Copy link
Contributor Author

z0ero commented Mar 9, 2022

ありがとうございます。 --atsumaru はニコ生ゲーム向けに適したオプションを勝手に使うオプションなので、 stiripbabel などと同じように常に true としてしまうのが良さそうです。

それも良さそうですが、他のオプションはユーザー指定の値を使っているようですし、minifyは必須要素ではないのであくまで指定できた方がよいと思います。強いて言うならデフォルトでminifyしてオプションで無効化できる方が自然かもしれません
こちらのPRではいったんこのままにしたいと思います。

@z0ero z0ero force-pushed the fix-export-atsumaru-minify branch from 1b3b919 to 92b3ef2 Compare March 9, 2022 01:38
@xnv
Copy link
Member

xnv commented Mar 9, 2022

なるほどです。将来的に opt-out にするなど挙動を変えるかもしれませんが、一旦この PR はそのままマージさせていただきます。

@xnv xnv merged commit 729ede6 into akashic-games:master Mar 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants