github / github/eslint-plugin-github

New rule: Ban top-level querying in module.

Đang mở
#228 0 bình luận 0 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
JavaScript
Star
336
Fork
67
Merge trung bình
1 ngày 9 giờ
Pull request đã merge (30 ngày)
6

Mô tả

Querying should be wrapped in event listeners such as `document.addEventListener('DOMContentLoaded', ...)` or a Web Component.

## 👎🏻 Examples of bad code.

```js
const el = document.querySelector('.my-element')
```

## 👍🏻 Example of good code.

```js
document.addEventListener('DOMContentLoaded', function() {
const el = document.querySelector('.my-element')
})
```

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

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Issue không nêu tên tệp, các bài kiểm thử hay các entry point. Hãy bắt đầu bằng cách tìm các định nghĩa quy tắc ESLint hiện có trong repository và các bài kiểm thử của chúng, sau đó so sánh cách chúng phát hiện các truy vấn ở cấp cao nhất và báo cáo các vi phạm. Công việc được xem là hoàn tất khi quy tắc mới từ chối truy vấn ở cấp module được minh họa, đồng thời vẫn cho phép truy vấn bên trong một listener DOMContentLoaded hoặc Web Component.

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
tooling
Loại issue
Tính năng
Độ khó
3/5
Thời gian dự kiến
1-2 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
45/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.