Skip to content

chore: exclude tests from all Codacy engines #16

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

Merged
cchacs merged 1 commit into from
Oct 1, 2026
Merged

chore: exclude tests from all Codacy engines #16

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions .codacy.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
---
exclude_paths:
- "tests/**"

@kilo-code-bot kilo-code-bot Bot Oct 1, 2026

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

SUGGESTION: Rationale references a file that does not exist in this repo, and the exclusion is broader than needed

The PR description justifies this by tests/js/screen.test.js calling new Function(...), but the only tracked file under tests/ is tests/test_routes.py; there is no tests/js/ directory and no screen.test.js anywhere in the tree. Either the motivating finding came from a different branch/repo state, or the trigger no longer exists.

Also note that a top-level exclude_paths removes the whole tests/ tree from every Codacy engine (linters included), which is wider than the single eval/Function() failure described. Narrowing to the specific engine/rule, or keeping the top-level entry with a comment explaining why all of tests/ is excluded, would keep test coverage for unrelated rules.

Not blocking - the config itself is valid and matches Codacy's schema.

Reply with @kilocode-bot fix it to have Kilo Code address this issue.

engines:
bandit:
exclude_paths:
Expand Down
Loading