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

Pull Pylance with Pyright 1.1.390 #9543

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

bschnurr
Copy link
Member

@bschnurr bschnurr commented Dec 4, 2024

No description provided.

Copy link
Contributor

github-actions bot commented Dec 4, 2024

According to mypy_primer, this change doesn't affect type check results on a corpus of open source code. ✅

@rohith-coder-45
Copy link

the modules from pylance is not working even though the required module is downloaded

Comment on lines +22 to +24
// Use a magic comment to prevent webpack from creating an extra chunk for the dynamic import by default.
// An extra chunk will still be created if explicitly configured in the webpack config.
TOML = await import(/* webpackMode: "eager" */ 'smol-toml');

Choose a reason for hiding this comment

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

This phrasing is slightly confusing, it could almost be read like it's an instruction and not the addition that you've made when viewed outside the scope of this PR?

Maybe explain more about why the comment works, since that does seem magic or like it shouldn't work lol, maybe a link to the associated documentation?

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.

3 participants