Skip to content

Fix #1 Refactoring#8

Open
syusui-s wants to merge 40 commits intomasterfrom
refactoring
Open

Fix #1 Refactoring#8
syusui-s wants to merge 40 commits intomasterfrom
refactoring

Conversation

@syusui-s
Copy link
Copy Markdown
Owner

@syusui-s syusui-s commented Mar 25, 2018

Fix #1

  • クラス類のリファクタリング
  • テストの整備
  • TypeScriptの導入
    • テストに対してもTypeScriptが使えるように df8fded
  • addition_patternの実装(互換性のためにMUST
  • hyperx.d.tsを提供
  • FormElementに生えてない可能性のあるプロパティをどう認識させるか

@syusui-s syusui-s changed the title Refactoring Fix #1 Refactoring Mar 25, 2018
@musou1500
Copy link
Copy Markdown
Collaborator

presetがないと,babelが何もお仕事をしない気がするので雑に babel-preset-env 突っ込んだ

@musou1500
Copy link
Copy Markdown
Collaborator

musou1500 commented Mar 25, 2018

@syusui-s
しかし,これってプロダクションビルドすると先頭のメタデータ書いてるコメント消えてしまわない?

@syusui-s
Copy link
Copy Markdown
Owner Author

syusui-s commented Mar 25, 2018

しかし,これってプロダクションビルドすると先頭のメタデータ書いてるコメント消えてしまわない

それ思ってて、JSON読み込んで、先頭にコメント付ける雑なスクリプトでも書くかーってなっている
他にいい方法があるかもしれないけど...

と思って調べたら、BannerPlugin成るものがあるらしい

webpack/webpack#165
https://webpack.js.org/plugins/banner-plugin/
https://github.com/webpack/webpack/blob/master/test/configCases/plugins/banner-plugin-hashing/webpack.config.js

@musou1500
Copy link
Copy Markdown
Collaborator

musou1500 commented Mar 25, 2018

@syusui-s BannerPluginで良さげ👍
gitのタグ読み取ってバージョン生成してあげるとそれっぽい(オーバースペックかも知れない)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

設計周りをキレイにする+重い動作を省く

2 participants