Skip to content
This repository has been archived by the owner on Jan 30, 2024. It is now read-only.

Log lines start with a space #230

Closed
jonas-schievink opened this issue Jun 18, 2021 · 0 comments · Fixed by knurling-rs/defmt#535
Closed

Log lines start with a space #230

jonas-schievink opened this issue Jun 18, 2021 · 0 comments · Fixed by knurling-rs/defmt#535
Assignees
@jonas-schievink
Labels
difficulty: easy Pretty easy to solve priority: low Low priority for the Knurling team status: needs PR Issue just needs a Pull Request implementing the changes type: bug Something isn't working

Comments

@jonas-schievink
Copy link
Contributor 

(HOST) INFO  flashing program (21.77 KiB)
(HOST) INFO  success!
────────────────────────────────────────────────────────────────────────────────
 INFO  (1/10) running `split_filters`...
└─ integration::tests::__defmt_test_entry @ tests/integration.rs:4
 INFO  (2/10) running `basic_roundtrip`...
└─ integration::tests::__defmt_test_entry @ tests/integration.rs:4

This looks a bit ugly. It happens when no timestamp is in use. Might be caused by defmt-decoder.
@jonas-schievink jonas-schievink added type: bug Something isn't working difficulty: easy Pretty easy to solve status: needs PR Issue just needs a Pull Request implementing the changes labels Jun 18, 2021
@Urhengulas Urhengulas added the priority: low Low priority for the Knurling team label Jun 19, 2021
@jonas-schievink jonas-schievink self-assigned this Jul 16, 2021
@jonas-schievink jonas-schievink mentioned this issue Jul 16, 2021
Merged
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jonas-schievink jonas-schievink

Labels
difficulty: easy Pretty easy to solve priority: low Low priority for the Knurling team status: needs PR Issue just needs a Pull Request implementing the changes type: bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Don't print leading space when timestamp is absent
2 participants
@jonas-schievink @Urhengulas