Skip to content
This repository has been archived by the owner. It is now read-only.

Initial working spike of named routes #1508

Merged
merged 4 commits into from Apr 14, 2014
Merged

Conversation

@grumpydev
Copy link
Member

grumpydev commented Apr 10, 2014

Tests need adding and fixing, but core changes work (I think :P)

⚠️ ⚠️ ⚠️ ⚠️ ⚠️ ⚠️ @thecodejunkie ⚠️ ⚠️ ⚠️ ⚠️ ⚠️ ⚠️

Don't pull this in yet 😉

@jchannon
Copy link
Member

jchannon commented Apr 10, 2014

makes sense if we're going to be adding stuff to routes

@grumpydev grumpydev added this to the 0.23 milestone Apr 14, 2014
@grumpydev
Copy link
Member Author

grumpydev commented Apr 14, 2014

Should be done now, assuming Travis gives it the all clear // @thecodejunkie

thecodejunkie added a commit that referenced this pull request Apr 14, 2014
Initial working spike of named routes
@thecodejunkie thecodejunkie merged commit 54c2114 into NancyFx:master Apr 14, 2014
1 check passed
1 check passed
continuous-integration/travis-ci The Travis CI build passed
Details
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.