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

バージョン表記の変更とHonoka固有のコードの削除 #6

Merged
merged 4 commits into from
Jul 15, 2015

Conversation

windyakin
Copy link
Owner

  • バージョン表記がぐちゃぐちゃだったのできちんと管理を分けるようにした
  • Forkテーマが作りやすいようにHonoka固有のコードを削除した

他のForkテーマ作っている方々的にはこの変更はどうでしょうか。逆に面倒になるようであればマージしません。

・HonokaのバージョンとBootstrapのバージョンが入り交じっていたのでそれぞれ別けて表記できるようにした
 それぞれのバージョンが挿入されるタイミングが異なるので注意
 ・Honokaのバージョンはpackage.jsonを参照し,**Gruntでreplace:versionのタスク実行時に**挿入されます
 ・Bootstrapのバージョンはvariables.scssを参照し,**SCSSのビルド時に**挿入されます
・バージョン表記変更に伴うGruntfileの修正
・パッケージファイル生成時にpackage.jsonのnameを参照するようにした
・バナー部分のウェブサイトの表記等に関してもpackage.jsonから取ってくるように
@windyakin windyakin changed the title Honoka バージョン表記の変更とHonoka固有のコードの削除 Jul 15, 2015
@kubosho
Copy link
Contributor

kubosho commented Jul 15, 2015

LGTM

@ysakasin
Copy link
Contributor

素晴らしい変更だとおもいます 👍

windyakin added a commit that referenced this pull request Jul 15, 2015
バージョン表記の変更とHonoka固有のコードの削除
@windyakin windyakin merged commit ddc7110 into master Jul 15, 2015
@windyakin
Copy link
Owner Author

早速のご返信ありがとうございます。

mergeしました。

windyakin pushed a commit that referenced this pull request Dec 21, 2015
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.

3 participants