Documentation conflicts with object-property-newline setting?
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 2/5
- Thời gian dự kiến
- 1-3 giờ
- Mức phù hợp với người mới
- 42/100
- Loại issue
- Tài liệu
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- eslint, javascript
- Lĩnh vực
- documentation, tooling
Hướng nghiên cứu
Bắt đầu với packages/eslint-config-airbnb-base/rules/style.js tại cấu hình object-property-newline, sau đó so sánh tùy chọn của nó với các phần functions, arrays và object có liên kết trong style guide. Xác định liệu style guide nên giải thích tùy chọn này hay cấu hình nên được làm cho nhất quán, rồi ghi lại hành vi đã chọn và lý do.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
It seems like the documentation consistently promotes having a new line for each item in multiline lists, whether it be in a function invocation or in an array (both of which mention bad examples of not having each item on a new line).
And even the object examples (although not referring to this specific rule) seem to favor this approach.
With that said, what is the reasoning behind allowing object properties to all appear on the same line?
It would seem more consistent to remove that option for object-property-newline. But at the very least, the reasoning and/or use case should probably be documented.
- Ngôn ngữ chính
- JavaScript
- Star
- 148k
- Fork
- 26.5k
- 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 airbnb/javascript
-
Severity: Unhandled promise rejection in `whitespace-async.js` when ESLint async path is usedCó thể đã có người làm @bodapatisaikrishna đã nhận 28 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
airbnb/javascript#3237 · 6 bình luận ·
-
Inconsistent semicolon usage in examples (Arrays vs Functions)Có thể đã có người làm @Developer-shivamMishra đã nhận 18 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
airbnb/javascript#3152 · 4 bình luận ·
-
No error handling around execSync + JSON.parse in whitespace.js (ESLint 9 path)Có thể đã có người làm @dataCenter430 đã nhận 223 ngày trước. Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 48/100
airbnb/javascript#3238 · 8 bình luận ·
-
Upgrading eslint-plugin-react-hooksCó thể đã có người làm @weihongyu12 đã nhận 368 ngày trước. Đang mở
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 38/100
airbnb/javascript#3186 · 3 bình luận · 2 reaction ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
airbnb/javascript#3173 · 9 bình luận · 1 reaction ·
Tất cả issue của airbnb/javascript
Issue tương tự
-
Add: Tooncast brazil feedĐang mởcheck:passed feeds:add
Độ khó 2/5 Dưới một giờ Mức phù hợp với người mới 62/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 70/100
codex-team/editor.js#3037 ·
-
No Code Attached Yet
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
joomla/joomla-cms#48626 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
complexity: trivial good first issue impact: low
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 78/100
ueberdosis/tiptap#8522 ·
Maintainer thường phản hồi trong vòng 1 ngày
-
internal os specific
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
frostney/GocciaScript#1688 ·
Maintainer thường phản hồi trong vòng 1 ngày