Skip to content

add 8.0.0589 patch - #245

Closed
tyru wants to merge 1 commit into
masterfrom
add-patch-8.0.0589-to-vimmagazine
Closed

tyru wants to merge 1 commit into
masterfrom
add-patch-8.0.0589-to-vimmagazine

Conversation

@tyru

@tyru tyru commented May 1, 2017

Copy link
Copy Markdown
Member

8.0.0589 のパッチが記載されていなかった件、手動で追加しちゃいましたが大丈夫でしょうか?
大丈夫そうだったらマージお願いします。

@tyru

tyru commented May 2, 2017

Copy link
Copy Markdown
Member Author

なぜ Travis が落ちているのかわからない…orz

@koron

koron commented May 2, 2017 •

Copy link
Copy Markdown
Member

そこはスクリプトで追加されている & スクリプトの実行時刻に依存するので、あまり手動で追加したくありません。どこまで取ったかを記録している JSON も変更しないと、次回実行時に 0589 が再掲されます。

@tyru

tyru commented May 2, 2017

Copy link
Copy Markdown
Member Author

では5月号に入れるべきでしょうか? #243

@koron

koron commented May 2, 2017

Copy link
Copy Markdown
Member

私は5月号に入れるほうが楽です。しかし

  1. JSONも手動で修正した上で
  2. 5月号の実行結果が 0590 以降から出ることを確認し
  3. 来月以降も同様のことを行ってくれる

ならば「あり」で良いかなと。

@tyru

tyru commented May 3, 2017

Copy link
Copy Markdown
Member Author

なるほど、じゃあ私も5月号でいいです(入れた方がいいんじゃないかな?ぐらいの感覚だったので)。
ありがとうございました。

@tyru tyru closed this May 3, 2017
@tyru tyru mentioned this pull request May 3, 2017
@crazymaster
crazymaster deleted the add-patch-8.0.0589-to-vimmagazine branch July 25, 2017 11:25
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.

2 participants