search

brownplt / lambda-py

Other
58 stars 10 forks source link

Multiple inheritance #28

Closed amtriathlon closed 11 years ago

amtriathlon commented 11 years ago

These commits should solve #20 and #27, they are part of the cleanup of application desugaring (more to be done). Since they make little changes in several places perhaps would be useful an early merge.

jpolitz commented 11 years ago

Looks great, thanks!

Are there any tests that failed that now pass that we can make a note of with the merge?

amtriathlon commented 11 years ago

Hi Joe, there were not tests for these bugs, in part because they were evident looking at the code, I've added a couple in the last commit to exercise them.

jpolitz commented 11 years ago

Thanks for adding.

I mostly want the tests for documentary reasons; it's easier for someone looking over the repo's history to look at the tests to see what we were trying to accomplish than to look at the many lines of diff.

On Wed, Jan 16, 2013 at 7:50 AM, Alejandro Martinez < notifications@github.com> wrote:

Hi Joe, there were not tests for these bugs, in part because they were evident looking at the code, I've added a couple in the last commit to exercise them.

— Reply to this email directly or view it on GitHubhttps://github.com/brownplt/lambda-py/pull/28#issuecomment-12317012.