Make filename, stream, and handlers parameters of logging.basicConfig mutually-exclusive

[brianschubert]

For reference:

>>> logging.basicConfig(filename="foo.log", handlers=[])
Traceback (most recent call last):
  ...
ValueError: 'stream' or 'filename' should not be specified together with 'handlers'

>>> logging.basicConfig(filename="foo.log", stream=None)
Traceback (most recent call last):
  ...
ValueError: 'stream' and 'filename' should not be specified together

>>> logging.basicConfig(filename="foo.log", filemode="w")
>>> logging.basicConfig(handlers=[], filemode="w")
Traceback (most recent call last):
  ...
ValueError: Unrecognised argument(s): filemode

Source: https://github.com/python/cpython/blob/3.14/Lib/logging/__init__.py#L2005
Docs: https://docs.python.org/3/library/logging.html#logging.basicConfig

[brianschubert]

cc @hamdanal since you looked at this function recently

[srittau]

This sounds like a case where we could forbid filemode to prevent an error where users pass it, expecting to do something when it doesn't. But I don't have strong feelings either way.

[brianschubert]

Sounds reasonable. The same is actually true for encoding and errors as well (all three do nothing when filename is not specified), so I think I'll remove all three from the overloads that don't include filename

[srittau]

LGTM, with one potential comment.

[github-actions]

According to mypy_primer, this change has no effect on the checked open source code. 🤖🎉

[srittau]

Thanks!