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

Rename Expected to ExpectedLength #419

Merged
merged 3 commits into from
Sep 21, 2019
Merged

Rename Expected to ExpectedLength #419

merged 3 commits into from
Sep 21, 2019

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a(n) refactor

Description

Renamed uuid::parser::Expected to uuid::parser::ExpectedLength

Motivation

Expected was slightly ambiguous.

Tests

Current tests should pass as is. No new tests.

Related Issue(s)

N/A

Signed-off-by: Hunar Roop Kahlon <[email protected]>
@Dylan-DPC-zz
Copy link
Member

Need to run rustfmt

Signed-off-by: Hunar Roop Kahlon <[email protected]>
@kinggoesgaming kinggoesgaming changed the title Remove Expected to ExpectedLength Rename Expected to ExpectedLength Jul 28, 2019
@kinggoesgaming
Copy link
Member Author

@Dylan-DPC this should be ready to go

@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Jul 28, 2019
419: Rename Expected to ExpectedLength r=Dylan-DPC a=kinggoesgaming

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

# Description
Renamed `uuid::parser::Expected` to `uuid::parser::ExpectedLength`

# Motivation
`Expected` was slightly ambiguous.

# Tests
Current tests should pass as is. No new tests.

# Related Issue(s)
N/A

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

bors bot commented Jul 28, 2019

Build failed

@kinggoesgaming
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Jul 30, 2019
419: Rename Expected to ExpectedLength r=kinggoesgaming a=kinggoesgaming

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

# Description
Renamed `uuid::parser::Expected` to `uuid::parser::ExpectedLength`

# Motivation
`Expected` was slightly ambiguous.

# Tests
Current tests should pass as is. No new tests.

# Related Issue(s)
N/A

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

bors bot commented Jul 30, 2019

Build failed

@kinggoesgaming
Copy link
Member Author

we need to switch from rustfmt-preview to rustfmt component

@kinggoesgaming
Copy link
Member Author

bors r+

bors bot added a commit that referenced this pull request Sep 21, 2019
419: Rename Expected to ExpectedLength r=kinggoesgaming a=kinggoesgaming

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

# Description
Renamed `uuid::parser::Expected` to `uuid::parser::ExpectedLength`

# Motivation
`Expected` was slightly ambiguous.

# Tests
Current tests should pass as is. No new tests.

# Related Issue(s)
N/A

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

bors bot commented Sep 21, 2019

@bors bors bot merged commit 1b61cb2 into master Sep 21, 2019
@delete-merged-branch delete-merged-branch bot deleted the expected-length branch September 21, 2019 05:58
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.

2 participants