Skip to content

Accept integer TOML truncation limits - #14796

Closed
DresdenGman wants to merge 1 commit into
pytest-dev:mainfrom
DresdenGman:accept-toml-integer-truncation-limits
Closed

Accept integer TOML truncation limits#14796
DresdenGman wants to merge 1 commit into
pytest-dev:mainfrom
DresdenGman:accept-toml-integer-truncation-limits

Conversation

@DresdenGman

Copy link
Copy Markdown

Fixes #14675.

Register truncation_limit_lines and truncation_limit_chars as int | str. This accepts native TOML integers while preserving existing string-based configuration formats and -o overrides.

The truncation consumer already converts both values with int(). The regression test now runs the existing threshold cases against both INI and native TOML configuration.

Tests:

  • PYTHONPATH=src python -m pytest -q testing/test_assertion.py::TestTruncateExplanation::test_truncation_with_config

@psf-chronographer psf-chronographer Bot added the bot:chronographer:provided (automation) changelog entry is part of PR label Jul 28, 2026
@Pierre-Sassoulas

Copy link
Copy Markdown
Member

Dupe of #14692, the first thing an AI should do before doing those automated fixes deemed easy should be to check if something already exists. This is the 6th time an AI MR is opened for this; it's infuriating.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bot:chronographer:provided (automation) changelog entry is part of PR

Projects

None yet

Development

Successfully merging this pull request may close these issues.

truncation_limit_lines and truncation_limit_chars should accept integer values in .toml configs

2 participants