[CHORE]: Warn on config values that do not match the schema
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
- 77/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- javascript
- Lĩnh vực
- data-visualization
Hướng nghiên cứu
Bắt đầu trong src/plot_api/plot_api.js tại setPlotContext, sau đó đọc src/plot_api/plot_config.js để hiểu configAttributes và siêu dữ liệu valType của nó. Kiểm tra từng giá trị áp dụng được bằng Lib.validate và cảnh báo khi không khớp, đồng thời bỏ qua các thuộc tính valType: 'any'. Hoàn thành khi các giá trị có kiểu không hợp lệ tạo ra cảnh báo mà không ép kiểu hoặc thay thế giá trị.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Description
Validate config values in setPlotContext against configAttributes and call Lib.warn on a mismatch.
Why should this change be made?
src/plot_api/plot_config.js says valType and values are documentation only, so a bad config value lands in _context in silence and fails somewhere else later.
Full coercion would replace bad values with defaults, which silently changes charts that work today. A warning gives the diagnostic benefit with no behavior change, and the hit rate tells us whether coercion is worth it later.
The metadata supports it: every typed config attribute is self-consistent, with no dflt outside its own values.
Scope
src/plot_api/plot_api.js- insetPlotContext, runLib.validate(value, attr)per key and warn on false- Skip the
valType: 'any'attributes.Lib.validateaccepts anything for them.
- Ngôn ngữ chính
- JavaScript
- Star
- 18.3k
- Fork
- 2k
- Merge trung bình
- 2 ngày 10 giờ
- Pull request đã merge (30 ngày)
- 30
Hướng dẫn đóng góp
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 plotly/plotly.js
-
chore P1 plotly-internal size: 1 task
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 82/100
-
chore P3 plotly-internal size: 1 task
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
-
bug infrastructure P2
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 65/100
-
feature good for agent P2 size: 3
Độ khó 1/5 1-3 giờ Mức phù hợp với người mới 68/100
Tất cả issue của plotly/plotly.js
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
palladius/rails8-app-on-gcp#145 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
dotenvx/dotenv-vscode#139 ·
-
test-change-proposal
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
web-platform-tests/interop#1455 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
corsairdev/corsair#1764 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100