Skip to content

Comments

[ty] Inlay Hint edit follow up#21621

Merged
Gankra merged 2 commits intoastral-sh:mainfrom
MatthewMckee4:inlay-hint-follow-up
Nov 25, 2025
Merged

[ty] Inlay Hint edit follow up#21621
Gankra merged 2 commits intoastral-sh:mainfrom
MatthewMckee4:inlay-hint-follow-up

Conversation

@MatthewMckee4
Copy link
Contributor

@MatthewMckee4 MatthewMckee4 commented Nov 24, 2025

Summary

Don't allow edits of some more invalid syntax types.

Test Plan

Add a test for x = Literal['a'] (similar) to show we don't allow edits.

@MatthewMckee4 MatthewMckee4 marked this pull request as ready for review November 24, 2025 23:29
@astral-sh-bot
Copy link

astral-sh-bot bot commented Nov 24, 2025

Diagnostic diff on typing conformance tests

No changes detected when running ty on typing conformance tests ✅

@astral-sh-bot
Copy link

astral-sh-bot bot commented Nov 24, 2025

mypy_primer results

No ecosystem changes detected ✅

No memory usage changes detected ✅

@AlexWaygood AlexWaygood added server Related to the LSP server ty Multi-file analysis & type inference labels Nov 24, 2025
@AlexWaygood AlexWaygood changed the title Inlay Hint edit follow up [ty] Inlay Hint edit follow up Nov 25, 2025
@MichaReiser MichaReiser requested review from Gankra and removed request for AlexWaygood, carljm, dcreager and sharkdp November 25, 2025 07:46
Copy link
Contributor

@Gankra Gankra left a comment

Choose a reason for hiding this comment

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

Awesome, thanks!

@Gankra Gankra merged commit 88bfc32 into astral-sh:main Nov 25, 2025
43 checks passed
carljm added a commit to mtshiba/ruff that referenced this pull request Nov 25, 2025
* main:
  [ty] Implement `typing.override` (astral-sh#21627)
  [ty] Avoid expression reinference for diagnostics (astral-sh#21267)
  [ty] Improve autocomplete suppressions of keywords in variable bindings
  [ty] Only suggest completions based on text before the cursor
  Implement goto-definition and find-references for global/nonlocal statements (astral-sh#21616)
  [ty] Inlay Hint edit follow up (astral-sh#21621)
  [ty] Implement lsp support for string annotations (astral-sh#21577)
  [ty] Add 'remove unused ignore comment' code action (astral-sh#21582)
  [ty] Refactor `CheckSuppressionContext` to use `DiagnosticGuard` (astral-sh#21587)
  [ty] Improve several "Did you mean?" suggestions (astral-sh#21597)
  [ty] Add more and update existing projects in `ty_benchmark` (astral-sh#21536)
  [ty] fix ty playground initialization and vite optimization issues (astral-sh#21471)
@MatthewMckee4 MatthewMckee4 deleted the inlay-hint-follow-up branch December 27, 2025 00:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

server Related to the LSP server ty Multi-file analysis & type inference

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants