this looks good. I marked 'Needs Fixing', just for a few nit picks.
I'd like some sort of unit test too.
The trend for adding unit tests is really to make 'handle' do less and move things into methods and use those methods.
« Back to merge proposal
this looks good. I marked 'Needs Fixing', just for a few nit picks.
I'd like some sort of unit test too.
The trend for adding unit tests is really to make 'handle' do less and move things into methods and use those methods.