search

mcollina / bloomrun

A js pattern matcher
MIT License
120 stars 10 forks source link

Beyond insertion order, match elements with more elements first #24

Closed mcollina closed 8 years ago

mcollina commented 8 years ago

This should bring bloomrun almost on par with patrun: it allows to match patterns with more properties before the others.

@mcdonnelldean please review.

Fixes #23.

mcdonnelldean commented 8 years ago

@mcollina This and Regex both look good. I need both of these for Varo to work (I can't control insertion order in react, it needs to be depth). Is this considered 2.0.0 or 1.1.0. I want to do a release.

mcollina commented 8 years ago

We can do 1.1.0, no breaking changes, only new features. Check the docs that are fully updated (with regexp and the depth bit)

Matteo Il giorno ven 27 nov 2015 alle 23:32 Dean McDonnell < notifications@github.com> ha scritto:

@mcollina https://github.com/mcollina This and Regex both look good. I need both of these for Varo to work (I can't control insertion order in react, it needs to be depth). Is this considered 2.0.0 or 1.1.0. I want to do a release.

— Reply to this email directly or view it on GitHub https://github.com/mcollina/bloomrun/pull/24#issuecomment-160222615.

mcdonnelldean commented 8 years ago

Awesome,

I'll get this out later today.

On 28 Nov 2015, at 02:19, Matteo Collina notifications@github.com wrote:

We can do 1.1.0, no breaking changes, only new features. Check the docs that are fully updated (with regexp and the depth bit)

Matteo Il giorno ven 27 nov 2015 alle 23:32 Dean McDonnell < notifications@github.com> ha scritto:

@mcollina https://github.com/mcollina This and Regex both look good. I need both of these for Varo to work (I can't control insertion order in react, it needs to be depth). Is this considered 2.0.0 or 1.1.0. I want to do a release.

— Reply to this email directly or view it on GitHub https://github.com/mcollina/bloomrun/pull/24#issuecomment-160222615.

— Reply to this email directly or view it on GitHub.