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

enable adapter tests for encode_{lower, upper} #390

Merged
merged 3 commits into from
Apr 1, 2019
Merged

enable adapter tests for encode_{lower, upper} #390

merged 3 commits into from
Apr 1, 2019

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a(n) other

Description

encode_{lower, upper} tests were not enabled when we enabled the functions.

Motivation

More tests

Tests

New tests enabled

Related Issue(s)

@kinggoesgaming kinggoesgaming added this to the 0.8.0 milestone Mar 31, 2019
@kinggoesgaming kinggoesgaming self-assigned this Mar 31, 2019
@kinggoesgaming kinggoesgaming requested a review from a team March 31, 2019 18:55
Dylan-DPC-zz
Dylan-DPC-zz previously approved these changes Mar 31, 2019
src/lib.rs Outdated Show resolved Hide resolved
Signed-off-by: Hunar Roop Kahlon <[email protected]>
Signed-off-by: Hunar Roop Kahlon <[email protected]>
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Apr 1, 2019
390: enable adapter tests for encode_{lower, upper} r=Dylan-DPC a=kinggoesgaming

**I'm submitting a(n)** other

# Description
`encode_{lower, upper}` tests were not enabled when we enabled the functions.

# Motivation
More tests

# Tests
New tests enabled

# Related Issue(s)


Co-authored-by: Hunar Roop Kahlon <[email protected]>
@bors
Copy link
Contributor

bors bot commented Apr 1, 2019

@bors bors bot merged commit 6cbfb3a into master Apr 1, 2019
@delete-merged-branch delete-merged-branch bot deleted the enable-tests branch April 1, 2019 07:17
bors bot added a commit that referenced this pull request Oct 17, 2019
433: Prepare for 0.8.0 release r=Dylan-DPC a=KodrAus

Closes #431 

[Changeset since the last release](0.7.4...master)

Includes:

- #427 
- #419 
- #424 
- #418 
- #413 
- #407 
- #404 
- #400 
- #399 
- #398 
- #397 
- #396 
- #394 
- #393 
- #390
- #389 
- #388 

I've also added some docs to the 128bit integer methods to clarify how they're different than `to_fields` (for consistency with other uuid libs in the wild).

Co-authored-by: Ashley Mannix <[email protected]>
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.

3 participants