Florian Best
Florian Best
```python log.error('hello "foo"') # OK log.error('hell o') # OK log.error('hello', extra={r'foo bar': 'bar'}) # OK log.error('hello', extra={r'foo': 'foo bar'}) # OK log.error('hello', extra={'foo': '"foo"'}) # OK log.error('hello', extra={'foo': "'foo'"}) #...
I have the same issue with a different library. I would like to just add a `# noqa: PLR6301` to the whole class: ```python3 class Command(BaseCommand): # noqa: PLR6301 def...
→ PR for `ignore_keys` in https://github.com/josheppinette/python-logfmter/pull/43
It's the https://reuse.software/spec-3.3/ specification. I just added them because my ruff complained about `missing-copyright-notice (CPY001)`. I don't see a real need for it, so I could also strip them away...
My test already compares the format with the parsed result.
I can easily split the tests case and make the round trip: `assert parse_logfmt(Logfmter.format_params(expected), **kwargs) == expected` but I currently already also verify the exact resulting string. And then I...
@td-anne I added two more additional round trip tests
@josheppinette I rebased it. But I don't think it's complete. We should speak about some behavior: 1. Should we allow `'` single quotes? logfmt only produces double quotes. I disabled...
5. I am not sure about compatibility with unquoted backslash sequences. The external compatibility test fails due to my demo change in 72e906065ef3b500f8f739fba48f06713f9cf62e. For example: logfmt currently produces: ```pycon >>>...
```bash $ echo 'foo="\\" bar=baz' | ./golang-logfmt-echo foo=\ bar=baz ```