Elvis Pranskevichus

Results 331 comments of Elvis Pranskevichus

Yeah, state `2` in the message refers to an unpropagated error state (i.e. "error consume"), I'll try to reproduce.

I don't think we're very special here, as storing credentials in plain text by default is something that devtools do regularly, take any CLI from `awscli` to `docker` to `gh`....

Are you able to provide steps to reproduce the crash or the crash dump?

There is already a space there, your terminal is just very bad at properly calculating the emoji character width: ![image](https://github.com/edgedb/edgedb-cli/assets/347119/bde5eec6-8cc3-41a0-b345-712bb42c40b3)

To be precise, this is an limitation on ad-hoc introspection of object types. A scalar global can be introspected without a problem with `introspect(typeof`.

Allowing free objects as input is, indeed, the way to go. We need to implement [typing free objects](https://github.com/edgedb/rfcs/blob/master/text/1022-freetypes.rst#typing-free-objects) from RFC 1022 first. Other prerequisites are in place, I think.

Error presentation (ISE) is a bug, but the actual issue is that there is not a thing named `null` in EdgeDB. You want `not exists .parent_revision` instead.

For reference, see earlier discussions: #180 and #5259. IMO, now that we have support for splats, we should probably look into using that machinery instead. For example we could make...

On a second thought, splats might not be ideal here as they'd pick up properties not specified in the `typeof` shape, so :thinking: