Skip to content

Reconfigured Pyright#1099

Merged
tisnik merged 1 commit intolightspeed-core:mainfrom
tisnik:reconfigured-pyright
Feb 3, 2026
Merged

Reconfigured Pyright#1099
tisnik merged 1 commit intolightspeed-core:mainfrom
tisnik:reconfigured-pyright

Conversation

@tisnik
Copy link
Contributor

@tisnik tisnik commented Feb 3, 2026

Description

Reconfigured Pyright

Type of change

  • Refactor
  • New feature
  • Bug fix
  • CVE fix
  • Optimization
  • Documentation Update
  • Configuration Update
  • Bump-up service version
  • Bump-up dependent library
  • Bump-up library or tool used for development (does not change the final image)
  • CI configuration change
  • Konflux configuration change
  • Unit tests improvement
  • Integration tests improvement
  • End to end tests improvement

Tools used to create PR

  • Assisted-by: N/A
  • Generated by: N/A

Summary by CodeRabbit

Release Notes

No user-facing changes in this release.

This update includes internal development infrastructure adjustments.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 3, 2026

Walkthrough

Removed three endpoint modules from the Pyright type checker's exclude configuration in pyproject.toml. The modules src/app/endpoints/query.py, src/app/endpoints/streaming_query.py, and src/utils/endpoints.py will now be subject to type validation where they were previously excluded.

Changes

Cohort / File(s) Summary
Type Checking Configuration
pyproject.toml
Removed three endpoint modules from Pyright's exclude list, enabling type checking for src/app/endpoints/query.py, src/app/endpoints/streaming_query.py, and src/utils/endpoints.py.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Reconfigured Pyright' accurately describes the main change: removing Pyright exclusions from the pyproject.toml configuration.
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

Important

Action Needed: IP Allowlist Update

If your organization protects your Git platform with IP whitelisting, please add the new CodeRabbit IP address to your allowlist:

  • 136.113.208.247/32 (new)
  • 34.170.211.100/32
  • 35.222.179.152/32

Reviews will stop working after February 8, 2026 if the new IP is not added to your allowlist.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

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

@tisnik tisnik merged commit f4dcb1c into lightspeed-core:main Feb 3, 2026
21 of 22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant