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

refactor v3 implementation #215

Merged
merged 3 commits into from May 3, 2018
Merged

refactor v3 implementation #215

merged 3 commits into from May 3, 2018

Conversation

kinggoesgaming
Copy link
Member

Signed-off-by: Hunar Roop Kahlon hunar.roop@gmail.com

I'm submitting a ...

  • bug fix
  • feature enhancement
  • deprecation or removal
  • refactor

Description

Move v3 implementation to the v3 module

Motivation

Part of the grand refactor

Tests

All tests, including v3 tests passing

Related Issue(s)

#124

Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request May 3, 2018
215: refactor v3 implementation r=Dylan-DPC a=kinggoesgaming

Signed-off-by: Hunar Roop Kahlon <hunar.roop@gmail.com>

<!--
    As we are working towards a stable version of uuid, we require that you 
    open an issue, before submitting a pull request. If the pull request is 
    imcomplete, prepend the Title with WIP: 
-->

**I'm submitting a ...**
  - [ ] bug fix
  - [ ] feature enhancement
  - [ ] deprecation or removal
  - [x] refactor

# Description
Move v3 implementation to the v3 module

# Motivation
Part of the grand refactor

# Tests
All tests, including v3 tests passing 

# Related Issue(s)
#124 


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

bors bot commented May 3, 2018

@bors bors bot merged commit d314352 into breaking May 3, 2018
@kinggoesgaming kinggoesgaming deleted the refactor/v3 branch May 3, 2018 21:10
LinusU pushed a commit to LinusU/uuid that referenced this pull request Feb 26, 2020
This was previously fixed in uuidjs/uuid@880d24e ,
but was regrtessed in README_js.md and hence in the newly generated README.md

Also reran `npm run-script prepare` to update README.md from the corrected version
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.

None yet

2 participants