Skip to content

Allow pylsp and pyrefly to work with this repository#2955

Merged
rwols merged 2 commits into
mainfrom
chore/pylsp-pyrefly
Jun 18, 2026
Merged

Allow pylsp and pyrefly to work with this repository#2955
rwols merged 2 commits into
mainfrom
chore/pylsp-pyrefly

Conversation

@rwols

@rwols rwols commented Jun 14, 2026

Copy link
Copy Markdown
Member

This adds a few lines of configuration so that:

  • pylsp does not complain about line length being >79 chars
  • pyrefly works

Comment thread setup.cfg Outdated
Comment thread pyproject.toml
Comment on lines +154 to +161
[tool.pyrefly]
python-version = "3.8.0"
site-package-path = ["./stubs"]
infer-with-first-use = false

[tool.pyrefly.errors]
bad-override = "ignore"
missing-source = "ignore"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I would be fine with introducing those as a means of evaluating different type checkers (there is also ty). I'm not super confident that pyright has a bright future.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

That said, please adapt the configuration so that there is not a ton of errors when running pyrefly check. Ignoring errors in tests and third_party would be a start.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This is just to remove some initial friction when browsing the repository with a pyrefly config enabled. But I could add some disabled checks. I actually haven’t ran pyrefly in lint mode from the CLI for this. Nor do I think we should engage further in adding this to CI.

@netlify

netlify Bot commented Jun 18, 2026

Copy link
Copy Markdown

Deploy Preview for sublime-lsp ready!

Name Link
🔨 Latest commit 821b3d4
🔍 Latest deploy log https://app.netlify.com/projects/sublime-lsp/deploys/6a33d6d648ca080008469725
😎 Deploy Preview https://deploy-preview-2955--sublime-lsp.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@rwols rwols merged commit 99b43f9 into main Jun 18, 2026
9 checks passed
@rwols rwols deleted the chore/pylsp-pyrefly branch June 18, 2026 19:37
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.

2 participants