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 the u128 feature #393

Merged
merged 3 commits into from
Apr 4, 2019
Merged

remove the u128 feature #393

merged 3 commits into from
Apr 4, 2019

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a(n) refactor|removal

Description

Removes the u128 feature. u128 related API is available as is now. Did not

Motivation

2018 edition now supports u128s out of the box

Tests

Current tests should pass. No need for new tests

Related Issue(s)

closes #366

Signed-off-by: Hunar Roop Kahlon <[email protected]>
@kinggoesgaming kinggoesgaming added breaking This causes a breaking change and will be merged to breaking branch. Release Notes labels Apr 4, 2019
@kinggoesgaming kinggoesgaming added this to the 0.8.0 milestone Apr 4, 2019
@kinggoesgaming kinggoesgaming self-assigned this Apr 4, 2019
@kinggoesgaming kinggoesgaming requested a review from a team April 4, 2019 01:51
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Apr 4, 2019
393: remove the u128 feature r=Dylan-DPC a=kinggoesgaming

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

# Description
Removes the `u128` feature. `u128` related API is available as is now. Did not 

# Motivation
`2018` edition now supports `u128`s out of the box

# Tests
Current tests should pass. No need for new tests

# Related Issue(s)
closes #366 


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

bors bot commented Apr 4, 2019

@bors bors bot merged commit 2be0546 into master Apr 4, 2019
@delete-merged-branch delete-merged-branch bot deleted the remove-u128-feature branch April 4, 2019 12:57
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
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.

remove u128 and const_fn feature gates
3 participants