remove `Uuid::new` function #214

Merged

kinggoesgaming merged 1 commit from rm-new into breaking

2018-05-02 00:52:12 -05:00

kinggoesgaming commented

2018-05-01 11:32:08 -05:00

(Migrated from github.com)

I'm submitting a ...

bug fix
feature enhancement
deprecation or removal
refactor

Description

Remove Uuid::new function and relevant tests

Motivation

Uuid::new is slightly unsafe since it has different behavior depending on the features enabled and the library user not checking for other results. Uuid::new_* should instead be used.

Tests

Remove test for Uuid::new

#176

**I'm submitting a ...** - [ ] bug fix - [ ] feature enhancement - [x] deprecation or removal - [ ] refactor # Description Remove `Uuid::new` function and relevant tests # Motivation `Uuid::new` is slightly unsafe since it has different behavior depending on the features enabled and the library user not checking for other results. `Uuid::new_*` should instead be used. # Tests Remove test for `Uuid::new` # Related Issue(s) #176

radix (Migrated from github.com) reviewed

2018-05-01 11:32:08 -05:00

KodrAus (Migrated from github.com) reviewed

2018-05-01 11:32:08 -05:00

Dylan-DPC-zz (Migrated from github.com) approved these changes

2018-05-02 00:42:50 -05:00

Dylan-DPC-zz commented

2018-05-02 00:43:39 -05:00

(Migrated from github.com)

bors: r+

bors[bot] commented

2018-05-02 00:52:11 -05:00

(Migrated from github.com)

Build succeeded

# Build succeeded * [continuous-integration/travis-ci/push](https://travis-ci.org/uuid-rs/uuid/builds/373777128?utm_source=github_status&utm_medium=notification) * [continuous-integration/appveyor/branch](https://ci.appveyor.com/project/KodrAus/uuid/build/1.0.472)