search

justinian / dice

Golang dice library
MIT License
20 stars 13 forks source link

Add Int() int function to RollResult interface #7

Closed frenata closed 7 years ago

frenata commented 7 years ago

Addresses the issue I raised a long while back. I'd implemented this a while back, but the recent PR made me decide to stop being lazy and send my own PR.

Of note is that in addition to implementing the requested functionality, I expanded test coverage across the package.

There were only a few minor fixes necessary to merge with the recent changes and keep the tests passing.

justinian commented 7 years ago

Awesome, thank you!

frenata commented 7 years ago

I just realized I neglected to update the ReadMe documentation to track with this change, apologies. I'll try to send a further PR along to fix it accordingly in a few days.