Skip to content

Conversation

@yusefnapora-sf
Copy link

Run with python3 roman.py <inputfile>

@kevsmith
Copy link
Owner

0/5 tests pass

@yusefnapora-sf
Copy link
Author

@kevsmith mind trying again? I tried to save a few chars but broke things :)

@kevsmith
Copy link
Owner

5/5 tests passed

@yusefnapora-sf
Copy link
Author

@kevsmith if you can run the tests again that would be sweet - I may have flown too close to the sun with the last few commits 😄

@yusefnapora-sf
Copy link
Author

I realized there was a bug in the last commit that would take more than two chars to fix, so I reverted it... I'm gonna call this done at 195 chars :)

@yusefnapora-sf
Copy link
Author

Okay, I lied - I got it down to 189 by mucking about with the list manipulation stuff.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants