Skip to content

Use improved tests required-filename pattern - #836

Open
CarrotManMatt wants to merge 4 commits into
mainfrom
improve-builtin-prek-hooks
Open

CarrotManMatt wants to merge 4 commits into
mainfrom
improve-builtin-prek-hooks

Conversation

@CarrotManMatt

Copy link
Copy Markdown
Member

This makes it so that all files within private (underscored) python modules within the tests/ directory are ignored. (Private files within public directories are not ignored but are approved by the existing pattern ^_.*\.py$.)

Also reorders builtin prek hooks to be in alphabetical order.

Signed-off-by: Matt Norton <matt@carrotmanmatt.com>
@CarrotManMatt CarrotManMatt self-assigned this Sep 15, 2026
@CarrotManMatt
CarrotManMatt requested a review from a team September 15, 2026 17:43
@CarrotManMatt CarrotManMatt added the deployment Changes to the deployment or CI/CD configuration label Sep 15, 2026
@codecov

codecov Bot commented Sep 15, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

@CarrotManMatt CarrotManMatt added awaiting committee review PRs awaiting approval from a CSS committee member sync Request bots to automatically keep this PR up to date with it's base branch labels Sep 15, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting committee review PRs awaiting approval from a CSS committee member deployment Changes to the deployment or CI/CD configuration sync Request bots to automatically keep this PR up to date with it's base branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants