LuaLS / LuaLS/lua-language-server
False negative: Warnings missing for table length mismatch
Chưa có ai nhận issue này.
- Ngôn ngữ chính
- Lua
- Star
- 4.4k
- Fork
- 442
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Mô tả
How are you using the lua-language-server?
NeoVim
Which OS are you using?
Linux
What is the issue affecting?
Type Checking, Diagnostics/Syntax Checking
Expected Behaviour
In the reproduction code below, f({0}) and f({0, 0, 0}) should issue warnings because they do not match the annotation [integer, integer], which requires a two-element table (tuple).
Actual Behaviour
There are no warnings issued for tables that have an invalid length.
I can see other warnings (e.g., for f("string")), so Neovim and other diagnostics are working correctly.
Reproduction steps
Open a text editor with lua-ls enabled and copy/paste this code.
---@alias Cursor [integer, integer]
---@param c Cursor
local function f(c) end
f("string")
f({ 0 })
f({ 0, 0, 0 })
Additional Notes
I tested with lua-language-server 3.9.1 and 3.13.2 and both have this problem.
Log File
I checked ~/.local/state/nvim/lsp.log since :LspInfo indicates it is the log file, but it was empty.
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Hướng nghiên cứu
Bắt đầu với đoạn mã tái hiện và đường dẫn kiểm tra kiểu cùng chẩn đoán của lua-language-server cho các bí danh bảng. So sánh cảnh báo hiện có cho f("string") với các cảnh báo còn thiếu cho f({ 0 }) và f({ 0, 0, 0 }). Hoàn tất khi các chú thích tuple báo cáo một chẩn đoán nếu độ dài bảng khác với hai phần tử bắt buộc.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Đánh giá
- Công nghệ
- lua
- Lĩnh vực
- devtools
- Loại issue
- Lỗi
- Độ khó
- 3/5
- Thời gian dự kiến
- 1-2 ngày
- Mức độ hoạt động
- Đình trệ
- Độ rõ ràng
- Khá rõ ràng
- Mức phù hợp với người mới
- 48/100