Revert 'require' word optimization
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 35/100
- Loại issue
- Tái cấu trúc
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- tooling
Hướng nghiên cứu
Bắt đầu bằng cách xác định tối ưu hóa regex được thảo luận trong issue và lần theo mối liên hệ của nó với syntax-error. Đọc issue browserify được tham chiếu và so sánh branch sm-fast được đề cập ở đó. Công việc được xem là hoàn tất khi tối ưu hóa được hoàn tác theo cách hỗ trợ việc loại bỏ syntax-error upstream, đồng thời các kiểm tra hiện có của dự án vẫn vượt qua.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
@zertosh What do you think about reverting the regex optimization so that syntax-error can be removed upstream?
Giant libraries with no require calls should be noParse'd anyway, and the performance difference, while significant, is only really marginal. Whereas the performance difference gleaned by removing syntax-error upstream is asymptotic and enormous.
IMO the bundle time of incremental re-builds is more important than one off full rebuilds, since incrementals are executed constantly (and also could be cached by browserify-incremental if you wanted).
I'm requesting this per your comment in this thread:
https://github.com/substack/node-browserify/issues/1208#issuecomment-92146245
If you did this and merged in your sm-fast branch, we could have really awesome rebuild times I think.
- Ngôn ngữ chính
- JavaScript
- Star
- 414
- Fork
- 59
- 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 browserify/detective
-
Update to [email protected]Có thể làm lại được Pull request cho issue này đã bị đóng mà không được merge. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 45/100
browserify/detective#88 · 5 reaction ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 48/100
browserify/detective#83 · 1 reaction ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 30/100
browserify/detective#71 · 5 bình luận · 2 reaction ·
-
Support ES6 importsĐang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
browserify/detective#56 · 3 bình luận · 4 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
browserify/detective#22 · 3 bình luận · 2 reaction ·
Tất cả issue của browserify/detective
Issue tương tự
-
Add: Film Mania PLĐang mởcheck:passed feeds:add
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/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 62/100
Maintainer thường phản hồi trong vòng 1 ngày
-
Add resource: BlockVectraĐang mở
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 76/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 75/100
SACGF/variantgrid#2791 ·
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 82/100
hiero-ledger/hiero-sdk-js#4761 ·
Maintainer thường phản hồi trong vòng 1 ngày