Skip to content

Comments

[ty] Ignore __all__ for document and workspace symbol requests#21928

Merged
BurntSushi merged 1 commit intomainfrom
ag/ignore-all-in-symbol-visitor
Dec 12, 2025
Merged

[ty] Ignore __all__ for document and workspace symbol requests#21928
BurntSushi merged 1 commit intomainfrom
ag/ignore-all-in-symbol-visitor

Conversation

@BurntSushi
Copy link
Member

We also ignore names introduced by import statements, which seems to
match pylance behavior.

Fixes astral-sh/ty#1856

We also ignore names introduced by import statements, which seems to
match pylance behavior.

Fixes astral-sh/ty#1856
@astral-sh-bot
Copy link

astral-sh-bot bot commented Dec 11, 2025

Diagnostic diff on typing conformance tests

No changes detected when running ty on typing conformance tests ✅

@astral-sh-bot
Copy link

astral-sh-bot bot commented Dec 11, 2025

mypy_primer results

Changes were detected when running on open source projects
scikit-build-core (https://github.com/scikit-build/scikit-build-core)
+ src/scikit_build_core/build/wheel.py:98:20: error[no-matching-overload] No overload of bound method `__init__` matches arguments
- Found 41 diagnostics
+ Found 42 diagnostics

No memory usage changes detected ✅

@AlexWaygood AlexWaygood added server Related to the LSP server ty Multi-file analysis & type inference labels Dec 11, 2025
@BurntSushi BurntSushi merged commit 0181568 into main Dec 12, 2025
44 checks passed
@BurntSushi BurntSushi deleted the ag/ignore-all-in-symbol-visitor branch December 12, 2025 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

server Related to the LSP server ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

symbol search returns only partial list of matches

3 participants