-
Notifications
You must be signed in to change notification settings - Fork 116
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
Use test-unit gem instead of test-framework of ruby repo #216
Conversation
Thanks. |
@kou What can I do the next step? If you are okay to this, I will merge this.
It's great for me. |
Could you wait for a few days? I want to review all changes and try this on local. For example, I think that we should not remove |
It's ok, no problem.
👍 I picked this helper from ruby/ruby repo originally. It's better to adjust to CSV library. |
|
Thanks! |
I replaced test-framework provided by ruby/ruby repo to test-unit gem and the additional assertions named
CoreAssertions
.This change makes unifying to assertion verb with
test-unit
gem.