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.

Sign up for GitHub

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

Jump to bottom

Topic/add routes #1

Closed

karenetheridge wants to merge 2 commits into rjbs:master from karenetheridge:topic/add_routes

Contributor

karenetheridge commented Apr 12, 2013

Minor doc edits (a Router::Dumb::Route object must be passed to add_route*)
add_route, add_route_unless_exists now handle a list of routes

karenetheridge added 2 commits

April 12, 2013 15:24


          support passing multiple routes at once to add_route, add_route_unles…

0e823ea

…s_exists


          fix docs - a Router::Dumb::Route object must be created

4e538a1

Owner

rjbs commented Apr 13, 2013

Thanks! I have applied the doc fix.

add_route takes only one to keep the interface simple and leave room for extra args later. I suggest:

$r->add_route($_) for @routes;

rjbs closed this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet