Skip to content

Comments

[ty] Rename functionArgumentNames to callArgumentNames inlay hint setting#19911

Merged
dhruvmanila merged 1 commit intomainfrom
dhruv/rename-inlay-hint-setting
Aug 14, 2025
Merged

[ty] Rename functionArgumentNames to callArgumentNames inlay hint setting#19911
dhruvmanila merged 1 commit intomainfrom
dhruv/rename-inlay-hint-setting

Conversation

@dhruvmanila
Copy link
Member

Summary

This PR renames ty.inlayHints.functionArgumentNames to ty.inlayHints.callArgumentNames which would contain both function calls and class initialization calls i.e., it represents a generic call expression.

@dhruvmanila dhruvmanila requested a review from carljm as a code owner August 14, 2025 08:46
@dhruvmanila dhruvmanila added the internal An internal refactor or improvement label Aug 14, 2025
@dhruvmanila dhruvmanila added the configuration Related to settings and configuration label Aug 14, 2025
@dhruvmanila dhruvmanila added server Related to the LSP server ty Multi-file analysis & type inference labels Aug 14, 2025
@github-actions
Copy link
Contributor

Diagnostic diff on typing conformance tests

Changes were detected when running ty on typing conformance tests
--- old-output.txt	2025-08-14 08:48:27.288427486 +0000
+++ new-output.txt	2025-08-14 08:48:27.353427566 +0000
@@ -1,5 +1,5 @@
 WARN ty is pre-release software and not ready for production use. Expect to encounter bugs, missing features, and fatal errors.
-fatal[panic] Panicked at /home/runner/.cargo/git/checkouts/salsa-e6f3bb7c2a062968/918d35d/src/function/execute.rs:215:25 when checking `/home/runner/work/ruff/ruff/typing/conformance/tests/aliases_typealiastype.py`: `infer_definition_types(Id(c104)): execute: too many cycle iterations`
+fatal[panic] Panicked at /home/runner/.cargo/git/checkouts/salsa-e6f3bb7c2a062968/918d35d/src/function/execute.rs:215:25 when checking `/home/runner/work/ruff/ruff/typing/conformance/tests/aliases_typealiastype.py`: `infer_definition_types(Id(2f8d6)): execute: too many cycle iterations`
 _directives_deprecated_library.py:15:31: error[invalid-return-type] Function always implicitly returns `None`, which is not assignable to return type `int`
 _directives_deprecated_library.py:30:26: error[invalid-return-type] Function always implicitly returns `None`, which is not assignable to return type `str`
 _directives_deprecated_library.py:36:41: error[invalid-return-type] Function always implicitly returns `None`, which is not assignable to return type `Self@__add__`

@github-actions
Copy link
Contributor

mypy_primer results

No ecosystem changes detected ✅
No memory usage changes detected ✅

@dhruvmanila dhruvmanila merged commit 1167ed6 into main Aug 14, 2025
39 checks passed
@dhruvmanila dhruvmanila deleted the dhruv/rename-inlay-hint-setting branch August 14, 2025 08:51
dcreager added a commit that referenced this pull request Aug 14, 2025
* main:
  Feature/build riscv64 bin (#19819)
  [ty] Add caching to `CodeGeneratorKind::matches()` (#19912)
  [ty] Rename `functionArgumentNames` to `callArgumentNames` inlay hint setting (#19911)
  [ty] Default `ty.inlayHints.*` server settings to true (#19910)
  [ty] Remove py-fuzzer skips for seeds that are no longer slow (#19906)
  [ty] fix deferred name loading in PEP695 generic classes/functions (#19888)
  [ty] Add some additional type safety to `CycleDetector` (#19903)
  [`flake8-blind-except`] Fix `BLE001` false-positive on `raise ... from None` (#19755)
  [ty] resolve docstrings for modules (#19898)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

configuration Related to settings and configuration internal An internal refactor or improvement 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.

1 participant