Skip to content
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.

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

Inconsistent module paths in symbol names when using from a.b import C syntax #133

Open
varungandhi-src opened this issue Nov 16, 2023 · 0 comments
Labels

Comments

@varungandhi-src
Copy link
Contributor

https://github.com/sourcegraph/scip-python/blob/scip/packages/pyright-scip/snapshots/output/nested_items/src/importer.py#L4-L6

from foo.bar import InitClass
#    ^^^^^^^ reference  snapshot-util 0.1 `foo.bar`/__init__:
#                   ^^^^^^^^^ reference  snapshot-util 0.1 `src.foo.bar`/InitClass#

The package name used should be consistent, so it should either be src.foo.bar or foo.bar based on the correct language semantics (e.g. if someone wrote src.foo.bar instead and that would work, then it should be src.foo.bar).

Originally flagged by @Titou325 here: #91 (comment)

@varungandhi-src varungandhi-src added the bug Something isn't working label Nov 16, 2023
@varungandhi-src varungandhi-src changed the title Incorrect symbol names for __init__ Inconsistent module paths in symbol names when using from a.b import C syntax Nov 16, 2023
@mmanela mmanela added the Migrated label May 6, 2024 — with Linear
@eseliger eseliger removed the Migrated label May 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants