Skip to content

Migrate toplevel tool configs to pyproject.toml#1741

Merged
rolandwalker merged 1 commit intomainfrom
RW/migrate-tool-config-to-pyproject-toml
Mar 25, 2026
Merged

Migrate toplevel tool configs to pyproject.toml#1741
rolandwalker merged 1 commit intomainfrom
RW/migrate-tool-config-to-pyproject-toml

Conversation

@rolandwalker
Copy link
Copy Markdown
Contributor

Description

  • migrate tox.ini to pyproject.toml sections
  • migrate pytest.ini to pyproject.toml sections

Motivation: tidy the toplevel directory.

Checklist

  • I added this contribution to the changelog.md file.
  • I added my name to the AUTHORS file (or it's already there).
  • To lint and format the code, I ran
    uv run ruff check && uv run ruff format && uv run mypy --install-types .

 * migrate tox.ini to pyproject.toml sections
 * migrate pytest.ini to pyproject.toml sections
@rolandwalker rolandwalker self-assigned this Mar 25, 2026
@github-actions
Copy link
Copy Markdown

  1. Regression: pytest config section is invalid in pyproject.toml
    In pyproject.toml, the PR adds [tool.pytest], but pytest reads ini-style options from [tool.pytest.ini_options].
    Impact: addopts = ['--ignore=mycli/packages/paramiko_stub/__init__.py'] will likely be ignored, so test discovery/collection behavior changes from current pytest.ini behavior.
    Fix: rename section to:
    [tool.pytest.ini_options]
    addopts = ["--ignore=mycli/packages/paramiko_stub/__init__.py"]

No other correctness/security issues stood out in this PR.
Residual risk: I couldn’t run pytest locally in this environment (pytest not installed), so this is static review only.

@rolandwalker rolandwalker merged commit 25e5e19 into main Mar 25, 2026
10 checks passed
@rolandwalker rolandwalker deleted the RW/migrate-tool-config-to-pyproject-toml branch March 25, 2026 20:04
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.

1 participant