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

Replace deprecated vital#of({plugin-name}) with vital#{plugin-name}#new() #435

Merged
merged 7 commits into from Aug 9, 2016

Conversation

tyru
Copy link
Member

@tyru tyru commented Jun 4, 2016

Close #432
Ref #431, #436

やったこと

  • doc/*.txt に大量の vital#of('vital') がありましたが、それは vital#{plugin-name}#new() に変えました。

@tyru
Copy link
Member Author

tyru commented Jun 4, 2016

Vim 7.4.1890 で Data.Closure のテストが失敗してる…

@thinca
Copy link
Member

thinca commented Jun 4, 2016

現時点では vital#{plugin-name}#new() はまだ存在していないのではないですか?

@tyru
Copy link
Member Author

tyru commented Jun 5, 2016

あー PR の順番がおかしいってそういうことですか。
正直この PR と #436 は issue に対応する形で出したかったというのと、2つ同時だと diff が大きくなるという理由で2つに分けました。
ただ単体でマージされるのは考えていなかったので、master にマージされるまでは順番がおかしいとか整合性がないのはあると思います。

@thinca
Copy link
Member

thinca commented Jun 5, 2016

分けるのは構わないですが、できればコミット順は正しくしてほしいです。

@thinca
Copy link
Member

thinca commented Jul 2, 2016

ping

Rename vital#{plugin-name}#of() to vital#{plugin-name}#new()
@thinca
Copy link
Member

thinca commented Aug 9, 2016

あれ、update branch 的なボタンってありませんでしたっけ…。(master をマージしないと CI 通らない)
ないなら手元でやるしかないかな。

@thinca thinca merged commit bff0d8c into master Aug 9, 2016
@thinca thinca deleted the revital-doc branch August 9, 2016 16:02
@tyru
Copy link
Member Author

tyru commented Aug 9, 2016

ありがとうございます!

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.

None yet

2 participants