Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Bug fixes, new setting, code clean up, and added support u64::max values per arg #408

Merged
merged 13 commits into from Feb 3, 2016

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented Feb 2, 2016

This turned into a little more than just a quick bug fix 😜

Thanks to #409 I've piggy backed on that and added support for > 256 values as well.

There's also just a bunch of code deduping and super small macro cleanups.

After this merges I'll put out 2.0.3

@yo-bot
Copy link

yo-bot commented Feb 2, 2016

r? @sru

(yo-bot has picked a reviewer for you, use r? to override)

@kbknapp kbknapp changed the title Minor bug fixes one new setting Bug fixes, new setting, code clean up, and added support u64::max values per arg Feb 2, 2016
@kbknapp
Copy link
Member Author

kbknapp commented Feb 2, 2016

88% coverage! Almost to 90!

@sru
Copy link
Contributor

sru commented Feb 3, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Feb 3, 2016

📌 Commit c31c2fb has been approved by sru

homu added a commit that referenced this pull request Feb 3, 2016
Bug fixes, new setting, code clean up, and added support u64::max values per arg

This turned into a little more than just a quick bug fix 😜

Thanks to #409 I've piggy backed on that and added support for > 256 values as well.

There's also just a bunch of code deduping and super small macro cleanups.

After this merges I'll put out 2.0.3
@homu
Copy link
Contributor

homu commented Feb 3, 2016

⌛ Testing commit c31c2fb with merge e47f825...

homu added a commit that referenced this pull request Feb 3, 2016
Bug fixes, new setting, code clean up, and added support u64::max values per arg

This turned into a little more than just a quick bug fix 😜

Thanks to #409 I've piggy backed on that and added support for > 256 values as well.

There's also just a bunch of code deduping and super small macro cleanups.

After this merges I'll put out 2.0.3
@homu
Copy link
Contributor

homu commented Feb 3, 2016

☀️ Test successful - status

@homu homu merged commit c31c2fb into master Feb 3, 2016
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.

None yet

4 participants