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

This branch was successfully deployed

1 active deployment
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