-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
Issue #7458: Nested $and not working in pouchdb-find. #7505
Conversation
This looks really good. I just need a little time to go over it before merging. Can you give me a few days and I'll get back to you on this. |
This looks good, thank you. Could you fix the linting issue https://travis-ci.org/pouchdb/pouchdb/jobs/448405064 and then I can merge once the tests are green. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just need the linting issue fixed.
Hi, fixed missing semicolon, will the tests run automatically again? |
yes they will
…On 05 Nov 2018, 12:26 +0200, BarboraNovotna ***@***.***>, wrote:
Hi, fixed missing semicolon, will the tests run automatically again?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.
|
I see that some tests failed but it seems that it is not related to my issue... Could you take a look at it? |
Sorry about that @BarboraNovotna, a change in travis broke our tests, could you possibly rebase and repush this and and can finally land once the tests are green, cheers |
Not looked into this a bunch, but looking at the failures I am guessing the same problem that I just seen in #7496, we use es5 in pouch and it seems that es6 looks like it breaks the build, I can see a const in there so probably it |
"docker: Error response from daemon: driver failed programming external connectivity on endpoint vigorous_shockley" - looks like env issue |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great, thanks
The failure was an unrelated intermittent. Apologies for the delay here, this is longer than we would like to be getting stuff merged but your PR came right at the same time as holidays + some test breakages Everythings good now, thanks for your contribution |
No description provided.