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

Add .match(router) method that returns a route without invoking a callback #9

Merged

Conversation

marcbachmann
Copy link
Contributor

This pull request contains a first part of #8 without introducing any big changes. We'll first need to release choojs/wayfarer#64 to get that working with the published module.

@yoshuawuyts
Copy link
Member

This looks good once wayfarer lands! :D

@marcbachmann marcbachmann force-pushed the add-match-method-of-wayfarer branch from 60fcd66 to d253163 Compare December 29, 2017 22:23
@marcbachmann
Copy link
Contributor Author

marcbachmann commented Dec 29, 2017

@yoshuawuyts I updated the wayfarer dependency. So this would be ready to merge.
I'll also quickly do the PR for the api change mentioned in #12 . It's up to you whether you want to release this using a minor bump.

edit: here's the pr for the api change: #13. It also includes this commit at the moment. So you can decide whether you want to merge this one first, do a minor bump or directly merge the other one with the major bump.

Copy link
Member

@yoshuawuyts yoshuawuyts left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rad!

@yoshuawuyts yoshuawuyts merged commit e7e6fda into choojs:master Jan 2, 2018
@yoshuawuyts
Copy link
Member

v2.2.0 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants