Jeff Wendling

Results 62 comments of Jeff Wendling

this would be great. an easy but janky way to do it is to require that the type not have any `.`s in it (because the parsing is harder) and...

i think the quoted version would be easiest to support with the current parser. also, this works nicely as long as conversions are applicable, but uuid is a special case...

this seems great. it should only require changing the template files, so we could even try it outside of the repo first.

one of the things about the Field types is that the zero value is invalid (that's why there's both a _set and a _null), and you have to use the...

comments are hard. i tried recently to add comment support to the ast and failed. i don't remember why it was so hard, but one property that is challenging is...

that's much easier to do

I finally checked my emails and saw this PR. I haven't taken a detailed look yet (I'll try to do that tomorrow), but I think this might live better in...

The debug logging is really for internal debugging when tests fail. I don't think there's useful data for external consumption.

The context returned from any stream has the transport available by using https://github.com/storj/drpc/tree/main/drpcctx#func--transport You're not the first one to have trouble finding this. It should definitely be documented better. Do...

saw a failure for this test today (with 16933 already) https://build.dev.storj.tools/blue/organizations/jenkins/storj-gerrit-verify/detail/storj-gerrit-verify/22567/tests ``` chore_test.go:1070: Error Trace: /tmp/workspace/storj-gerrit-verify/satellite/payments/accountfreeze/chore_test.go:1070 /tmp/workspace/storj-gerrit-verify/satellite/console/accountfreezes.go:905 /tmp/workspace/storj-gerrit-verify/satellite/satellitedb/consoledb/consoledb.go:118 /tmp/workspace/storj-gerrit-verify/satellite/satellitedb/dbx/gen.go:82 /tmp/workspace/storj-gerrit-verify/shared/dbutil/txutil/transactions.go:76 /tmp/workspace/storj-gerrit-verify/shared/dbutil/txutil/transactions.go:37 /tmp/workspace/storj-gerrit-verify/satellite/satellitedb/dbx/gen.go:81 /tmp/workspace/storj-gerrit-verify/satellite/satellitedb/consoledb/consoledb.go:104 /tmp/workspace/storj-gerrit-verify/satellite/console/accountfreezes.go:855 /tmp/workspace/storj-gerrit-verify/satellite/console/accountfreezes.go:916 /tmp/workspace/storj-gerrit-verify/satellite/payments/accountfreeze/chore.go:596 /tmp/workspace/storj-gerrit-verify/satellite/payments/accountfreeze/chore.go:97 /go/pkg/mod/storj.io/[email protected]/sync2/cycle.go:146 /tmp/workspace/storj-gerrit-verify/satellite/payments/accountfreeze/chore.go:91 /tmp/workspace/storj-gerrit-verify/private/lifecycle/group.go:87...