-
Notifications
You must be signed in to change notification settings - Fork 860
WIP: Add Pyrefly support #7567
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
base: main
Are you sure you want to change the base?
WIP: Add Pyrefly support #7567
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
All contributors have signed the CLA ✍️ ✅ |
for more information, see https://pre-commit.ci
|
Thanks @yangdanny97! We're happy to review this. If this gets merged, we can document its support in our own docs, but also on Pyrefly docs (here: https://pyrefly.org/en/docs/IDE-features/#notebook-support) — does that sound good to you? |
|
I have read the CLA Document and I hereby sign the CLA |
|
@akshayka hi, thanks! Yep, I'll update Pyrefly's docs once this is merged. I'm still working on debugging the diagnostics. Pyrefly is sending them (I can see it in the lsp logs) but somehow the listener does not pick them up, even if diagnostics box is checked in the user config. All other LSP functionalities work. |
for more information, see https://pre-commit.ci
|
I wonder if there's an issue with my local setup. I can't get diagnostics to appear for basedpyright or ty in my local dev build either, even without the changes in this PR. @winstxnhdw do you remember having to do any sort of special dev setup to test your PRs for bpr/ty, or were the steps in https://github.com/marimo-team/marimo/blob/main/CONTRIBUTING.md#setup sufficient? |
|
Hey @yangdanny97 I didn't have to do anything special. I did have to restart my setup every time I switch LSPs though. It might be worth looking at the console logs to see if an exception is being thrown. |
📝 Summary
Similar to recent PRs for ty & basedpyright, this PR adds support for Pyrefly, another fast language server and type checker
I read through the discussion in #1163, and I think that Pyrefly would be a good addition (perhaps falling somewhere between basedpyright & ty on the spectrum of maturity).
It's being used in similar tools like Positron, and is significantly more complete in terms of type system coverage than ty
Diagnostics
🔍 Description of Changes
📋 Checklist