Skip to content

[red-knot] update to latest Salsa with fixpoint caching fix#17179

Merged
carljm merged 3 commits intomainfrom
cjm/bumpsalsa2
Apr 3, 2025
Merged

[red-knot] update to latest Salsa with fixpoint caching fix#17179
carljm merged 3 commits intomainfrom
cjm/bumpsalsa2

Conversation

@carljm
Copy link
Contributor

@carljm carljm commented Apr 3, 2025

With this PR, we no longer "hang" (not actually a hang, just an explosion in execution time) when checking pylint.

@AlexWaygood AlexWaygood added the ty Multi-file analysis & type inference label Apr 3, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Apr 3, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@carljm carljm changed the title [WIP] try Salsa with fixpoint fix [red-knot] update to latest Salsa with fixpoint caching fix Apr 3, 2025
@carljm carljm marked this pull request as ready for review April 3, 2025 15:58
@MichaReiser
Copy link
Member

How long before @sharkdp adds new projects to our mypy primer run? :)

@AlexWaygood
Copy link
Member

I shall put in my bet at "7 minutes"

@carljm
Copy link
Contributor Author

carljm commented Apr 3, 2025

How long before @sharkdp adds new projects to our mypy primer run? :)

I think the projects that no longer hang still panic at the moment, that's next on my list...

@carljm carljm merged commit 45c4373 into main Apr 3, 2025
22 checks passed
@carljm carljm deleted the cjm/bumpsalsa2 branch April 3, 2025 16:05
@AlexWaygood
Copy link
Member

AlexWaygood commented Apr 3, 2025

Ah yes, I see we still panic on pylint with this fix landed (twice, in fact!), but we now promptly finish the process after panicking, whereas previously we panicked and then... hung around for ages before finishing the process 😄

Progress!

dcreager added a commit that referenced this pull request Apr 3, 2025
* origin/main:
  [red-knot] Add `Type::TypeVar` variant (#17102)
  [red-knot] update to latest Salsa with fixpoint caching fix (#17179)
  Upgrade to Rust 1.86 and bump MSRV to 1.84 (#17171)
  [red-knot] Avoid unresolved-reference in unreachable code (#17169)
  Fix relative import resolution in `site-packages` packages when the `site-packages` search path is a subdirectory of the first-party search path (#17178)
  [DO NOT LAND] bump Salsa version (#17176)
  [syntax-errors] Detect duplicate keys in `match` mapping patterns (#17129)
maxmynter pushed a commit to maxmynter/ruff that referenced this pull request Apr 3, 2025
…h#17179)

With this PR, we no longer "hang" (not actually a hang, just an
explosion in execution time) when checking pylint.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants