Skip to content
This repository was archived by the owner on Jul 19, 2020. It is now read-only.

Remove the Option wrapper from Route's state#206

Merged
hgzimmerman merged 3 commits into
masterfrom
remove_option_from_route
Dec 17, 2019
Merged

Remove the Option wrapper from Route's state#206
hgzimmerman merged 3 commits into
masterfrom
remove_option_from_route

Conversation

@hgzimmerman
Copy link
Copy Markdown
Member

@hgzimmerman hgzimmerman commented Dec 17, 2019

fixes #205

@hgzimmerman hgzimmerman added the breaking change This change will result in a broken build or differing behavior in end-user crates. label Dec 17, 2019
@hgzimmerman
Copy link
Copy Markdown
Member Author

hgzimmerman commented Dec 17, 2019

Breakage is done to both Route's state field type and to the arguments to Switch::from_route_part.

@hgzimmerman hgzimmerman merged commit 7837f76 into master Dec 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

No one assigned

Labels

breaking change This change will result in a broken build or differing behavior in end-user crates.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Route state field should be just T

1 participant

@hgzimmerman