iJackUA / rwdtow

Ruby Web Dev: The Other Way. Personal best practices guide.
http://rwdtow.stdout.in/
Other
265 stars 26 forks source link

Drop Scientist library from benchmark list #55

Open davydovanton opened 5 years ago

davydovanton commented 5 years ago

scientist library was created for safe refactoring parts of ruby code. It compares a result of old code and a result of new code, after calling some callbacks (like something wrong and something is okay). After that scientist returns old code result.

In this case, using scientist as a benchmark library is a strange idea. You can do it, but it's not a general case for it. That's why I think can drop it from

iJackUA commented 5 years ago

I agree that it's not a perfect case, but still, want to mention it somewhere and there is no better place in a book yet. Maybe just add a comment that it's not a perfect case?

davydovanton commented 5 years ago

good idea. let's update it