Frederick

Query builder

10

Something better than string concatenation for dynamic query construction.

The example error below should only suggest `edgedb.OptionalDuration` because the field is not required. ``` 2022/06/29 13:02:06 edgedb.BinaryProtocolError: building decoder from ParameterStatus state_description: expected state.config.durprop to be edgedb.Duration or edgedb.OptionalDuration...

see also https://github.com/edgedb/edgedb-go/issues/228#issuecomment-1167199853_

add custom formatting to errors

1

One nice feature of golang's most popular error library [pkg/errors](https://github.com/pkg/errors) is [custom formatting](https://pkg.go.dev/github.com/pkg/errors#hdr-Formatted_printing_of_errors) for printing errors. This is something that would improve the developer experience by associating stack traces with...

builder API for connection options

1

I think it would be nice to change the options/configuration API by adding options "constructors", giving the options type a builder API and making the option's fields private. This would:...

Standardize on a single marker name: "order"

3

Reading the documentation I noticed that there are a lot of potential marker names including run(), and even custom markers proposed in #10. It seems like this has the potential...

depends on https://github.com/edgedb/nebula/pull/14

edgedb instance link prompts for password twice

2

When the `--password` option is passed then `edgedb instance link` prompts twice for the password. It seems like the user should only be asked once. - EdgeDB CLI Version: EdgeDB...

Inconsistent help message formatting

1

The help for edgedb instance is formatted strangely when --help is passed but not for -h or help . I haven't found another command that has this format or difference...

DDL snip-it output from create-migration seems confusing

1

The DDL snip-its that are sometimes printed when a migration is created seems confusing. It feels like just that snip-it is supposed to mean something to me. Which is strange...