Skip to content

エントリを生成するRakefile#24

Merged
yasulab merged 4 commits intookinawarb:gh-pagesfrom
kimihito:rake-new-post
May 19, 2014
Merged

エントリを生成するRakefile#24
yasulab merged 4 commits intookinawarb:gh-pagesfrom
kimihito:rake-new-post

Conversation

@kimihito
Copy link
Contributor

rake new_postでやったことエントリを生成できるといいなぁと思ってやってみました。#17 の問題が解消できるかなと思ったりしています。コード汚いです、すいません。

@kimihito kimihito changed the title エントリを自動生成するRakefile エントリを生成するRakefile May 19, 2014
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tokiyomi gem を使うと、綺麗に書けそう 👍

Usage
Time.now       # => 2013-01-10T12:34:56
Tokiyomi.parse('10日前') # => 2012-12-31T12:34:56

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

そういうGemがあるんですね!!ありがとうございます!!

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

5/21から何日経過してるか調べてるのでtokiyomi使えない気がしました。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

rake new_postを実行した日を基準に、色々と処理したいのかなー、と。
なので、5/21は、とりあえずの(今回限りの)exampleというだけのような... (違ってたらごめんなさい 🙇 )。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

と思ったけど、よくよくコードをみるとそんなこと無かった! w

失礼しました...orz

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

💭 (第1回 Weekly Meetup の日を基準にして計算すると、「いつから始まったのか」という情報も付加できて良いかもしれないっすね...)

Facebook Eventのログに残っている最初のMeetupが第3回 Weekly Meetupで、それが 2012/02/29 に開催されたので、第1回 Weekly Meetup は 2012/02/15 かな?

とはいえ、休みの日もある(あった)と思うので、その際の例外処理も必要なのかな。

@hanachin
Copy link
Member

checkoutして動作も確認しました。
気になるところはコメントしましたが、このままマージして 🆗 です!

LGTM

もしもどこか修正するなら修正した後またレビューします。
よろしくおねがいします :shipit:

@hanachin
Copy link
Member

今実行したら前回開催の日付間違ってたけど

何回目のOkinawarb Meetupsに参加しましたか?(前回開催(2014-05-21):117回):
Switched to a new branch '2014-05-19-no117-hanachin'
Creating new post: _posts/2014-05-19-no117-hanachin.md

どうでもいい

めっちゃ便利なのでこれつかってPull Request作ってみます。

@hanachin
Copy link
Member

💭 あ、5/21以前だからこの動作なのか、すみません 🙇
ファイル名はコマンド実行した日の日付なんですねー。

@yasulab
Copy link
Member

yasulab commented May 19, 2014

#25 で動くことは確認できたので、とりあえずマージしますね。
リファクタリングや細かい修正などは、別途PR出しましょう 👍

Thx! @kimihito

yasulab added a commit that referenced this pull request May 19, 2014
エントリを生成するRakefile
@yasulab yasulab merged commit 82f0d32 into okinawarb:gh-pages May 19, 2014
@yasulab
Copy link
Member

yasulab commented May 19, 2014

PR出しました #26

@kimihito
Copy link
Contributor Author

いろいろコメントありがとうございました! #26 も助かります!PR出してよかったですー。ブランチ削除しまーす。

@kimihito kimihito deleted the rake-new-post branch May 20, 2014 05:35
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