Defaults added

[bharel]

closes #24

Pretty simple. Two unit tests to check for a defaults parameter alone, and with a rename.

[nhairs]

Overall looks good.

I'm wondering if we should add something to the quick start docs.

[nhairs]

Hi @bharel,

I plan to do a new point release as soon as #23 is ready (it is currently waiting on a new release of msgspec).

It would be good if this was ready to land in the same release.

[bharel]

Haven't run black on the last change, I apologize. Should be good now.

I'm receiving import errors on mypy, but those are unrelated (running on py3.13) where msgspec doesn't exist yet...?

************* Module pythonjsonlogger.msgspec
src/pythonjsonlogger/msgspec.py:20:0: E0401: Unable to import 'msgspec.json' (import-error)
************* Module pythonjsonlogger.orjson
src/pythonjsonlogger/orjson.py:20:0: E0401: Unable to import 'orjson' (import-error)

[nhairs]

I'm receiving import errors on mypy, but those are unrelated (running on py3.13) where msgspec doesn't exist yet...?

Yep that's fine - on CI we run it using 3.12 which I can see is passing 🎉

[nhairs]

I think we still need to add something to the "Quick start" docs, we also need to add to the doctstring change log.

~If I'm able to push to the branch I'll do these change myself to save on time. ~ I cannot push.

Could you add the following to docs/quickstart.md before the Static Fields section:

#### Default Fields

Default fields that are added to every log record prior to any other field can be set using the `default` argument.

```python
formatter = JsonFormatter(
    defaults={"remote_ip": None}
)
# ...
logger.info("this overwrites the remote_ip field", extras={"remove_ip": "1.1.1.1"})
```

And the following to the BaseFormatter docstring:

        *Changed in 3.2*: `defaults` argument is no longer ignored.

[bharel]

Oh you managed to push it I see :-)

Can we close this then? :D