Prevent parent scope symbols leaking into module's scope
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 20/100
Hướng nghiên cứu
Bắt đầu bằng cách lần theo việc tra cứu ký hiệu trong ngăn xếp scope và create_default_context(), sau đó kiểm tra cách FrameType và các hằng số như NU_LIB_DIRS được biểu diễn. Quyết định cách các frame Barrier và Prelude nên tương tác với việc cô lập module, các lệnh mặc định và các lớp phủ nupm. Công việc được xem là hoàn tất khi các quy tắc tra cứu và hành vi của NU_LIB_DIRS được đặc tả và triển khai, với coverage cho các ký hiệu của scope cha, các lệnh Prelude và việc import module.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Leaking user-defined symbols from a parent scope to a module's scope can lead to subtle bugs. The module's scope should be as isolated from the parent scope as possible, but not completely (e.g., we still need basic Nushell commands like ls to be visible).
Could be partially addressed by introducing two new FrameTypes:
Barrier: Stops symbol lookup when traversing the scope stack. This prevents the symbol lookup from searching the parent scopes.Prelude: With only Barrier, we'd lose “default” commands that we want to be visible (likels). Perhaps we could have a Prelude frame type which would get initialized first duringcreate_default_context()with these default commands.
The question is what to do with constants like NU_LIB_DIRS, e.g.,:
const NU_LIB_DIRS = $NU_LIB_DIRS | append '/home/spam/foo`
module {
use bar.nu # should it look for 'bar.nu' in '/home/spam/foo'?
}
We might want to decide on this based on the envisioned functionality of how nupm is supposed to use overlays as “virtual environments”, relying on modifying the NU_LIB_DIRS constant.
- Ngôn ngữ chính
- Rust
- Star
- 36
- Fork
- 11
- 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
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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.
Issue khác của nushell/new-nu-parser
-
Suppot parsing bulit-in keywordsĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
nushell/new-nu-parser#64 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
nushell/new-nu-parser#60 ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 42/100
nushell/new-nu-parser#59 ·
-
bug typechecker
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
nushell/new-nu-parser#53 · 11 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 35/100
nushell/new-nu-parser#47 · 3 bình luận ·
Tất cả issue của nushell/new-nu-parser
Issue tương tự
-
good first issue help wanted r-docs
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
servo/rust-smallvec#824 ·
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 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
priority middle
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 62/100
KATO-Hiro/AtCoderClans#12940 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 83/100
Maintainer thường phản hồi trong vòng 1 ngày
-
o8 next debug build aborts on Glass to Dark appearance switch (nil identifier messaged before check)Đang mởbug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
hurttlocker/o8#5112 ·