search

frc-frecon / frecon

An API for building scouting apps for FRC competitions
MIT License
3 stars 0 forks source link

Remove support for IRB. #63

Closed rye closed 9 years ago

rye commented 9 years ago

We no longer want to do this checking. IRB isn't very testable, and since pry is listed as a dependency for this project, we shouldn't use IRB (since it likely hasn't been used at all anyways). This pull request, when merged, should close #62.

Sammidysam commented 9 years ago

Just curious, what do you mean by IRB not being as testable?

rye commented 9 years ago

Some of the methods used in the setup of IRB require way too much work to stub them out. When testing IRB, we obviously don't want to be starting IRB instances for every example. Pry is so simple because we can just stub out the .pry method for a given class or object, which is much easier.