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

Fix #1 Refactoring #8

Open
wants to merge 40 commits into
base: master
Choose a base branch
from
Open

Fix #1 Refactoring #8

wants to merge 40 commits into from

Conversation

syusui-s
Copy link
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
Collaborator

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

@musou1500
Copy link
Collaborator

musou1500 commented Mar 25, 2018

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

@syusui-s
Copy link
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
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