angular / angular/dev-infra

Consider adding a wrapper around `jasmine_node_test` rule

Đang mở
#338 0 bình luận 2 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
JavaScript
Star
77
Fork
84
Merge trung bình
2 ngày 5 giờ
Pull request đã merge (30 ngày)
37

Mô tả

PR https://github.com/angular/angular/pull/44505 updated angular/angular repo to use ES2015 instead of ES5. The mentioned change revealed that there is a duplication in a lot of places where the `bootstrap` is defined, for example:

```
jasmine_node_test(
bootstrap = ["//tools/testing:node_no_angular_es2015"],
```
or:
```
jasmine_node_test(
bootstrap = ["//tools/testing:node_es2015"],
```

This makes further ES version upgrades quite error-prone (and time-consuming), since we need to change ~100 files.

The proposal is to create a couple wrappers, for example:
- ng_node_test
- no_ng_node_test

and specify `bootstrap` there by default (and allow overrides), so that it's easier to update ES version.

Hướng dẫn đóng góp

Chưa lập chỉ mục được hướng dẫn đóng góp cho kho mã nguồn này

Hướng nghiên cứu

Bắt đầu bằng cách xác định rule jasmine_node_test và các vị trí gọi bootstrap lặp lại được nêu trong issue. Định nghĩa các entry point ng_node_test và no_ng_node_test được đề xuất với các giá trị bootstrap mặc định và hỗ trợ override, sau đó xác minh rằng các vị trí gọi hiện có có thể sử dụng chúng và rằng các thay đổi phiên bản ES trong tương lai sẽ cần ít chỉnh sửa hơn.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
javascript
Lĩnh vực
build-system, testing
Loại issue
Tính năng
Độ khó
4/5
Thời gian dự kiến
3-5 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
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.