Closed reillysiemens closed 8 years ago
All tests don't pass? Intentional?
Add a couple of small comments. Potentially add me as a push member on your fork so I can do formatting and code tidying when you're not available?
Yes, all tests don't pass. The token tests are broken.
I'm going to make this a WIP/Don't merge PR instead. I'll change the title again when I feel it's ready.
@rawrgulmuffins: What small comments did you have in mind?
Reading PR now.
This PR was targeted to Red-Green-Refactor:master
. Needs to be targeted to red-green-refactor:reference_implementation
rather thenmaster
.
My mistake for merging it.
Paging @reillysiemens
Description
This set of changes includes improvements to error handling by raising the
CalcError
exception with useful error information. It also makes thetest_calculator.py
(formerlyintepreter.py
) tests pass and wires the calculator up to the console script entry point given insetup.py
. This allows the calculator to be invoked ascalc
after runningpip install -e .
.Check List