Skip to content

test: expect sorted __all__ after ruff autofix - #80

Merged
RonnyPfannschmidt merged 1 commit into
mainfrom
claude/fix-iniconfig-test-eukh2n
Oct 6, 2026
Merged

RonnyPfannschmidt merged 1 commit into
mainfrom
claude/fix-iniconfig-test-eukh2n

Conversation

@RonnyPfannschmidt

Copy link
Copy Markdown
Member

Requested by Ronny · project thread

🤖 Written by Claude Opus 5.5 via Claude Code for the iniconfig maintainers; I prompted it, it did the work, I read it.

Before: test_api_import fails on main because the pre-commit.ci autofix in #73 (ruff RUF022) sorted iniconfig.__all__, while the test still asserted the old order.

After: the test expects the sorted order, which ruff now enforces. Full suite (51 tests) and pre-commit run -a pass locally.

🤖 Generated with Claude Code

https://claude.ai/code/session_015GEmLmbkaC57DHre56mKTo


Generated by Claude Code

pre-commit.ci (#73) sorted iniconfig.__all__, breaking test_api_import.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015GEmLmbkaC57DHre56mKTo
@RonnyPfannschmidt RonnyPfannschmidt self-assigned this Oct 6, 2026
@RonnyPfannschmidt
RonnyPfannschmidt merged commit 48359c3 into main Oct 6, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants