Skip to content
This repository has been archived by the owner on Jun 27, 2022. It is now read-only.

Adhers to the specification #46

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Adhers to the specification #46

wants to merge 2 commits into from

Conversation

kettanaito
Copy link
Member

@kettanaito kettanaito commented Aug 22, 2021 •
edited
Loading

Changes

  • The library now adheres to the route parameters specification.
  • Adds path parsing step to reduce a path string to the list of tokens.
  • Supports optional parameters.
  • Supports zero-or-more parameters.
  • Supports one-or-more parameters.

@kettanaito kettanaito mentioned this pull request Aug 22, 2021
Closed
@kettanaito
Copy link
Member Author

The tests are failing because the multiple optional path parameters feature is not currently implemented.

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
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@kettanaito