Unintentionally global variables stat and ret in async_lib?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 1/5
- Thời gian dự kiến
- Dưới một giờ
- Mức phù hợp với người mới
- 55/100
Hướng nghiên cứu
Open lua/plenary/async_lib/util.lua around lines 292-302 and inspect how stat and ret are assigned and used. The issue is done when those names no longer unintentionally create or modify globals while preserving the existing async_lib behavior.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
It looks like stat and ret should be local in this code and not global.
This is a common mistake in lua that happens easily I guess. Found by lua language server autocomplete.
To make them local would make them encapsulated, and you avoid risk - people using the unintentionally used variables, or overwriting people's globals.
Link:
- Ngôn ngữ chính
- Lua
- Star
- 3.5k
- Fork
- 341
- Chỉ số merge pull request
- Không có pull request nào được merge trong 30 ngày
Chuẩn bị môi trường
Dự án này không cung cấp dev container, Dockerfile hay hướng dẫn đóng góp, nên bạn cần tự thiết lập môi trường: hãy bắt đầu từ README và xem hướng dẫn đóng góp lần đầu của chúng tôi để biết các bước chung.
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.
Issue khác của nvim-lua/plenary.nvim
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 88/100
nvim-lua/plenary.nvim#682 ·
-
uniform usage of `os.getenv`Có thể đã có người làm @rakotomandimby đã nhận 123 ngày trước. Đang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 72/100
nvim-lua/plenary.nvim#680 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
nvim-lua/plenary.nvim#675 ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 28/100
nvim-lua/plenary.nvim#672 · 1 bình luận ·
-
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
nvim-lua/plenary.nvim#671 · 1 bình luận ·
Tất cả issue của nvim-lua/plenary.nvim
Issue tương tự
-
Enhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
mogenson/PaperWM.spoon#178 ·
-
enhancement good first issue
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
-
ci: Verify install never runs examples_install or tests_install, so a clear they widen passesĐang mởseverity: low
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
luainkernel/lunatik#1876 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug ready ticket
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Domktorymysli/vclu#5 ·