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

adjust ParseError for indicating if an urn format was expected #397

Merged
merged 2 commits into from
Apr 18, 2019

Conversation

kinggoesgaming
Copy link
Member

I'm submitting a(n) refactor

Description

Allows for future parsing ability for the adapters.

Motivation

This was a breaking change as such was needed to be added in 0.8.0`

Tests

Current tests should pass. No new tests introduced.

Related Issue(s)

closes #370

@kinggoesgaming kinggoesgaming added the breaking This causes a breaking change and will be merged to breaking branch. label Apr 18, 2019
@kinggoesgaming kinggoesgaming added this to the 0.8.0 milestone Apr 18, 2019
@kinggoesgaming kinggoesgaming requested a review from a team April 18, 2019 02:00
@kinggoesgaming kinggoesgaming self-assigned this Apr 18, 2019
@Dylan-DPC-zz
Copy link
Member

bors: r+

bors bot added a commit that referenced this pull request Apr 18, 2019
397: adjust ParseError for indicating if an urn format was expected r=Dylan-DPC a=kinggoesgaming

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

# Description
Allows for future parsing ability for the adapters.

# Motivation
This was a breaking change as such was needed to be added in 0.8.0`

# Tests
Current tests should pass. No new tests introduced.

# Related Issue(s)
closes #370 


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

bors bot commented Apr 18, 2019

@bors bors bot merged commit 9182349 into master Apr 18, 2019
@delete-merged-branch delete-merged-branch bot deleted the urn-error branch April 18, 2019 09:50
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.

alter the uuid::parser::ParseError to properly account for urn syntax
2 participants