-
-
Notifications
You must be signed in to change notification settings - Fork 277
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 shindo #311
Comments
👍 I think this would be a good move. |
Also I'd be up for tackling some of this but if so I'd like to work in chunks as I think the burden of tracking along w/ other changes would lead to an unpleasant monster-merge. What do you think? |
Yeah. Doing it in chunks seems most tenable, especially in terms of other movement. That said, ideally we wouldn't have to support both suites for an extended period, as that has it's own nastiness. |
Agreed, I'm with you on that. |
@bjeanes a fine question. It is somewhat of both I think (change of preference and restatement). I'm leaning somewhat toward rspec due to it having a more active (and welcoming) community around it. I like the minimality of minitest, but not method-named-tests nor the relatively closed/unwelcoming community I've found there. Happy to discuss further though. |
Sounds good to me. I've been using minitest in new projects of mine for simplicity reasons, but RSpec is a fine choice given the mentioned constraints. |
@bjeanes yeah. I definitely know of pros/cons in both columns, but leaning toward rspec at least slightly at present due to more active development/community. |
Close in favor of #575 |
shindo is weird and a barrier to contribution/updates. Seems like it might be good to migrate. These days I'm somewhat leaning toward rspec with assertions (not #should).
The text was updated successfully, but these errors were encountered: