Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@pygarap
Copy link
Collaborator

@pygarap pygarap commented Oct 16, 2025

Important

Fix a bug where basedpyright and pyright as well as Zed editor could not parse our pyproject.toml file!

  • Simplified and condensed build backend documentation for clarity.
  • Streamlined tool.coverage.report.exclude_lines by grouping and reformatting patterns for readability and consistency.

This pull request updates the pyproject.toml file with improvements to documentation and configuration clarity, focusing on build backend explanation and coverage exclusion patterns. The changes streamline the build backend section and clarify coverage exclusion rules for maintainability and readability.

Build Backend Documentation Improvements:

  • Replaced a lengthy, detailed explanation of the build backend with a concise summary, making it easier for newcomers to understand the project's use of PEP 517/518 and editable installs.

Coverage Exclusion Pattern Clarifications:

  • Refactored the exclude_lines section for test coverage, grouping patterns by theme (pragmas, imports, typing, main guards, exceptions, and placeholders), simplifying comments, and improving consistency in regex formatting.

- Simplified and condensed build backend documentation for clarity.
- Streamlined `tool.coverage.report.exclude_lines` by grouping and reformatting patterns for readability and consistency.
@codecov
Copy link

codecov bot commented Oct 16, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@pygarap pygarap requested a review from pydanny October 16, 2025 00:41
Copy link
Member

@pydanny pydanny left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pydanny pydanny merged commit f858689 into main Oct 16, 2025
10 checks passed
@pydanny pydanny deleted the coverage_bug branch October 16, 2025 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants