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

Warn user about unsupported pull-syntax inside posh queries #29

Open
theronic opened this issue Mar 30, 2018 · 3 comments
Open

Warn user about unsupported pull-syntax inside posh queries #29

theronic opened this issue Mar 30, 2018 · 3 comments

Comments

@theronic
Copy link

Could Posh please warn the user when they try to pull inside a p/q posh query?

I lost two days trying to figure out why my ClojureScript threw Assert failed: (db? db) when posh does not support the pull syntax inside posh queries. It says so in the README - I've read it before. I knew this information, but I forgot. It was especially hard to debug from a React Native app where the error for using a pull query inside q looks like this:

image

@jamesnyika
Copy link

@theronic - I hear you man. I lost a week! But what I did to begin mitigating this kind of timewasting was to start my own documentation wiki in my repo. I began to document important lessons I learned as I build my application - specifically for things like this which are important reminders that are easy to forget. It then cuts down my context switching when I return to the code particularly after longer absences. That way I keep the lessons close to the codebase where I learned them. It is really slow moving but that is how I am paying the debt!

@mpdairy
Copy link
Owner

mpdairy commented Jul 18, 2018

Oh sorry, ya'll! I've been out of the Clojure world for some time (using Haskell) and didn't see this issue. Do you want it to give a runtime warning or maybe I could just make it bold in the Readme?

@theronic
Copy link
Author

Runtime warning would be best :)

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

No branches or pull requests

3 participants