airbnb / airbnb/javascript

`if` statements – one line vs. one expression

Đang mở
#445 5 bình luận 0 reaction 0 người được giao Xem trên GitHub
needs eslint rule change/addition question
Ngôn ngữ chính
JavaScript
Star
148k
Fork
26.6k
Chỉ số merge pull request
Không có pull request nào được merge trong 30 ngày

Mô tả

Until I started using ESLint with your configuration I saw nothing against the rules in this:

``` js
$ cat if.js
const [one, two] = [1, 2];

if (one !== two) throw new Error(
'One does not equal two'
);
```

But ESLint does:

``` sh
$ eslint if.js

if.js
3:0 error Expected { after 'if' condition curly

✖ 1 problem (1 error, 0 warnings)
```

Is this intended? In my opinion the pattern I’ve been using is explicit – and more readable than this:

``` js
if (one !== two) {
throw new Error(
'One does not equal two'
);
}
```

The parens form a visual brace-like block much like in #438.

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áo cáo xác định quy tắc `curly` của ESLint và cấu hình của dự án, kèm theo phần so sánh với dạng có dấu ngoặc nhọn và tham chiếu đến #438. Hãy bắt đầu bằng việc kiểm tra `curly` được cấu hình như thế nào và hướng dẫn kiểu mã đề cập đến các câu lệnh `if` một dòng ra sao; công việc được xem là hoàn tất khi xác định được liệu quy tắc hiện tại có phải là chủ ý hay không và ghi lại hoặc thay đổi hướng dẫn đó cho phù hợp.

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

Đánh giá

Công nghệ
eslint, javascript
Lĩnh vực
documentation, tooling
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
Cần làm rõ
Mức phù hợp với người mới
25/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.