Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @gnzlbg (or someone else) soon.

If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.

Please see the contribution instructions for more information.

also, if this looks good can i bump the verision

This LGTM. @bors: r+

I'll bump the version when doing the next release.

📌 Commit 83ccc7b has been approved by gnzlbg

⌛ Testing commit 83ccc7b with merge 48c1b13...

💔 Test failed - checks-travis

@bors: r+

💡 This pull request was already approved, no need to approve it again.

• This pull request previously failed. You should add more commits to fix the bug, or use retry to trigger a build again.

📌 Commit 83ccc7b has been approved by gnzlbg

⌛ Testing commit 83ccc7b with merge 5b989db...

☀️ Test successful - checks-cirrus, checks-travis, status-appveyor
Approved by: gnzlbg
Pushing 5b989db to master...

thanks