Linters / Python updates #35

Merged

yippee0903 merged 4 commits from dev into master

2026-02-26 22:03:39 +00:00

yippee0903 commented

2026-02-26 21:59:21 +00:00

(Migrated from github.com)

Summary by CodeRabbit

Chores
- Updated type checker configuration to exclude additional directories, improving build performance and reducing analysis scope.

## Summary by CodeRabbit * **Chores** * Updated type checker configuration to exclude additional directories, improving build performance and reducing analysis scope.

coderabbitai[bot] commented

2026-02-26 21:59:38 +00:00

(Migrated from github.com)

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

Configuration used: Path: .coderabbit.yaml

Review profile: ASSERTIVE

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5a2963044b and da46e8f183.

📒 Files selected for processing (1)

pyproject.toml

📝 Walkthrough

Walkthrough

Extends the pyright type checker exclusion list in pyproject.toml to skip node_modules, pycache, hidden directories, and venv directories from type checking, in addition to the existing data directory exclusion.

Changes

Cohort / File(s)	Summary
Pyright Configuration `pyproject.toml`	Extended the type checker exclude patterns to include `/node_modules`, `/__pycache__`, `*/.`, and `venv` alongside the existing `data` directory exclusion.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name	Status	Explanation	Resolution
Title check	❓ Inconclusive	The title is vague and overly broad. It uses generic phrasing like 'updates' and 'Linters / Python' without clearly specifying the primary change, which is extending the pyright type checker exclusion list.	Replace with a more specific title such as 'Expand pyright exclusion list for node_modules, cache, and venv' to clearly convey the main change.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

Create PR with unit tests
Post copyable unit tests in a comment
Commit unit tests in branch dev

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

No actionable comments were generated in the recent review. 🎉 <details> <summary>ℹ️ Recent review info</summary> **Configuration used**: Path: .coderabbit.yaml **Review profile**: ASSERTIVE **Plan**: Pro <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 5a2963044bbf547db37aec3c8612eda00971780b and da46e8f1832e9a0b75737efef53e90eb6180a913. </details> <details> <summary>📒 Files selected for processing (1)</summary> * `pyproject.toml` </details> </details> ---  <details> <summary>📝 Walkthrough</summary> ## Walkthrough Extends the pyright type checker exclusion list in pyproject.toml to skip node_modules, __pycache__, hidden directories, and venv directories from type checking, in addition to the existing data directory exclusion. ## Changes |Cohort / File(s)|Summary| |---|---| |**Pyright Configuration** `pyproject.toml`|Extended the type checker exclude patterns to include `**/node_modules`, `**/__pycache__`, `**/.*`, and `venv` alongside the existing `data` directory exclusion.| ## Estimated code review effort 🎯 1 (Trivial) | ⏱️ ~2 minutes </details>   <details> <summary>🚥 Pre-merge checks | ✅ 2 | ❌ 1</summary> ### ❌ Failed checks (1 inconclusive) | Check name | Status | Explanation | Resolution | | :---------: | :------------- | :------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------- | :---------------------------------------------------------------------------------------------------------------------------------------------- | | Title check | ❓ Inconclusive | The title is vague and overly broad. It uses generic phrasing like 'updates' and 'Linters / Python' without clearly specifying the primary change, which is extending the pyright type checker exclusion list. | Replace with a more specific title such as 'Expand pyright exclusion list for node_modules, cache, and venv' to clearly convey the main change. | <details> <summary>✅ Passed checks (2 passed)</summary> | Check name | Status | Explanation | | :----------------: | :------- | :--------------------------------------------------------------------------------------------------------- | | Description Check | ✅ Passed | Check skipped - CodeRabbit’s high-level summary is enabled. | | Docstring Coverage | ✅ Passed | No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. | </details> ✏️ Tip: You can configure your own custom pre-merge checks in the settings. </details>   <details> <summary>✨ Finishing Touches</summary> <details> <summary>🧪 Generate unit tests (beta)</summary> - [ ]  Create PR with unit tests - [ ]  Post copyable unit tests in a comment - [ ]  Commit unit tests in branch `dev` </details> </details>   --- Comment `@coderabbitai help` to get the list of available commands and usage tips.

coderabbitai[bot] (Migrated from github.com) approved these changes

2026-02-26 22:02:36 +00:00