Skip to content

Allow asserts and unused args in test files #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
msaipraneeth wants to merge 1 commit into
base: main
Choose a base branch
from
Open

Allow asserts and unused args in test files #45

msaipraneeth wants to merge 1 commit into from

Conversation

@msaipraneeth
Copy link
Contributor

@msaipraneeth msaipraneeth commented Dec 9, 2025

Summary for the PR:

  • Moved S101 (assert detection) from global ignore to per-file ignore for test files
  • Added ARG (unused args) to per-file ignore for test files to accommodate pytest fixtures

@github-actions
Copy link

github-actions bot commented Dec 9, 2025

Coverage

Coverage Report
File Stmts Miss Cover Missing
init.py 0 0 100%
example_transform.py 28 0 100%
example_workflow.py 36 2 94% 53 56
TOTAL 64 2 97%  

Tests Skipped Failures Errors Time
4 0 💤 0 ❌ 0 🔥 4.357 ⏱

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@msaipraneeth