Richard Janis Goldschmidt

Results 54 comments of Richard Janis Goldschmidt

Found this because I have the same predicament. Never had the issue in vim, and vscode comes with a config `file.insertFinalNewline` because they also don't do it by default.

To add to this: this means that users cannot perform a right-click when using the trackpad (because that's ctrl-click on macOS). Instead of opening the right-click menu to interact with...

This caught me today. I would love to see this released soon. 🙏

What is Celestia-Node's recommendation for client handling of blobs not being found? Is error code 1 only used for blobs not found? (IIRC no, the error code is used for...

I actually ran into this scenario: 1. `blob.GetAll(namespace: [foo]) => 1 blob`, 2. `blob.GetAll(namespace: [bar]) => Error(blob: not found)`, 3. `blob.GetAll(namespace: [foo, bar]) => Error(blob: not found)`. The Celestia Node...

Maybe the following Google guidelines for API design could be an inspiration: 1. https://google.aip.dev/132 (list methods) 2. https://google.aip.dev/231 (batch methods) Specifically the last point under guidance in AIP 231 is...

@Wondertan better docs are definitely desirable. Even better would be a dedicated (and documented, stable, supported) JSONRPC error code for `blob: not found` and potentially a structured error in the...

Where does the JSON-RPC API define this error? I'm a bit confused as to which API you're talking about. Celestia-Node officially documents a jsonrpc API, which is the one we...

@distractedm1nd I believe one way forward would could be to provide for example `blob.ListGetAll` and `blob.BatchGetAll`, where the former covers your use case, while `blob.BatchGetAll` would cover the current functionality.

Yeah, I was just looking at the source, and noticed that `opt_devtype = DEVTYPE_SINK;` in `main()`. I guess the structure of `GetDevice` in `pulse.cc` is dictated by the PulseAudio API...