Jonathan Pallant

Results 245 comments of Jonathan Pallant

This looks OK but it needs a wire-format bump so we need to wait until after 1.0 is released, when we have a bunch of stuff to wrap up into...

Still want to get this in, but the process will probably need to be: * Merge to main * Bump the Wire Format * Do a defmt-parser / defmt-decoder release...

You can do this: ```json { "version": "0.2.0", "configurations": [ { "cwd": "${workspaceRoot}", // TODO to debug a different program the app name ("hello") needs to be changed "executable": "./target/thumbv7em-none-eabihf/debug/panic",...

I think one solution for this is for the main branch to be changed to have the version number `N + 1` right after version `N` is released, and for...

Can we add in an option to put the logs back like they were? So anyone who doesn't want or like the new format can just add the option and...

I tried with probe-rs, which just uses the default format. Before: ```console $ DEFMT_LOG=debug cargo run --target=thumbv7em-none-eabi --bin levels -- --allow-erase-all Compiling defmt-macros v0.3.9 Compiling defmt v0.3.8 Compiling panic-probe v0.3.2...

They seem happy for it to go into 0.3.12, as they are pinned to `==0.3.11` anyway. @Dirbaio said: > I don't think i'd expect the cli output for logging to...

OK, I was testing the *wrong* version of probe-rs. I edited to correct the output. The problem with removing the newline is that the location field is of variable length,...

After a lot of discussion on the probe.rs matrix chat, we iterated to: ```rust const FORMAT: &str = "{[{L}]%bold} {s}"; const FORMAT_WITH_LOCATION: &str = "{[{L}]%bold} {s} {({c:bold}:{ff}:{l:1})%dimmed}"; const FORMAT_WITH_TIMESTAMP: &str...

Ah, there's an issue. `levels:bin/levels.rs:15` isn't clickable in VS Code. Maybe we should change the first `:` to a space.