Colin McDonnell

Results 258 comments of Colin McDonnell

Any reason we haven't merged this @jaclarke?

As I mentioned on Slack: I'm also happy just returning decimal values as strings (which can then be passed into decimal.js or any equivalent library, should the user need to...

Seems this is a problem with inferring cardinality from `e.cast`. Either of these approaches will work: ```ts e.cast(e.uuid, e.str("c900312b-2b8d-4d8d-b491-0761ecbb72e1")); // works e.uuid("c900312b-2b8d-4d8d-b491-0761ecbb72e1"); // works e.cast(e.uuid, "c900312b-2b8d-4d8d-b491-0761ecbb72e1"); // doesn't work ```...

Perhaps `e.op_all` or `e.op_many`? Agreed w.r.t `e.and` - retracted.

The easiest thing to do in this situation is check the query builder code into Git so you don't need to worry about regenerating it. We try to generally discourage...

This is a tricky one. It's a limitation of EdgeQL currently, and I can't find a simpler workaround than the one you're currently using. I suspect you'd have an easier...

> Personally, I prefer something close to Graphql's style: I don't hate this, it's familiar to people and has better autocompletion than `e.op`. But it also doesn't have the composability...

> do we have to use e.uuid()? Can we allow ID types to accept string literals directly? The casting seems a bit too verbose and not super necessary We are...

I already dislike `npx edgedb-migrate` and I like `edgedb-ci-migrate` even less. Hyphenated commands like this just aren't very elegant and feel too technical/hackery. I propose an alternative command naming scheme...

I think this is a great idea. @1st1 any thoughts? I like the `logging: boolean | LoggingAdapter` approach. Do you have a proposal for that interface? Super simple option: ```ts...