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

remove Uuid::from_random_bytes #399

Merged
merged 2 commits into from
Apr 21, 2019
Merged

remove Uuid::from_random_bytes #399

merged 2 commits into from
Apr 21, 2019

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a(n) removal

Description

Removed Uuid::from_random_bytes(crate::Bytes) -> Self

Motivation

Uuid::from_random_bytes was already deprecated. Time to remove.

Tests

All tests should pass. A test regarding the Uuid::from_random_bytes was removed.

Related Issue(s)

N/A

Use Builder instead

Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
@kinggoesgaming kinggoesgaming added the breaking This causes a breaking change and will be merged to breaking branch. label Apr 21, 2019
@kinggoesgaming kinggoesgaming self-assigned this Apr 21, 2019
@kinggoesgaming kinggoesgaming added this to the 0.8.0 milestone Apr 21, 2019
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Apr 21, 2019
399: remove Uuid::from_random_bytes r=Dylan-DPC a=kinggoesgaming

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

# Description
Removed `Uuid::from_random_bytes(crate::Bytes) -> Self`

# Motivation
`Uuid::from_random_bytes` was already deprecated. Time to remove.

# Tests
All tests should pass. A test regarding the `Uuid::from_random_bytes` was removed.

# Related Issue(s)
N/A

Co-authored-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
@bors
Copy link
Contributor

bors bot commented Apr 21, 2019

@bors bors bot merged commit 5d6b5e6 into master Apr 21, 2019
@delete-merged-branch delete-merged-branch bot deleted the remove-from-random_bytes branch April 21, 2019 21:31
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 <ashleymannix@live.com.au>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This causes a breaking change and will be merged to breaking branch.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants