noprompt / meander

Tools for transparent data transformation
MIT License
928 stars 55 forks source link

meander.interpreter.epsilon/finder doesn't work with multiple clauses #229

Open prepor opened 2 years ago

prepor commented 2 years ago

Hi.

[query f & more-clauses] args version of finder calls finder-from with a collection of pairs while finder-from expects clauses list and partition it by itself.

And there is no single test for meander.interpreter.epsilon/finder with multiple clauses.

The fix is simple: Instead of (finder-from (cons [query f] (partition 2 more-clauses))) we can (finder-from (cons query (cons f more-clauses)))

I can prepare a PR if it would help

noprompt commented 2 years ago

I can prepare a PR if it would help

Yes. Please!