Add numeric and others validations for options
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 5/5
- Thời gian dự kiến
- Hơn một tuần
- Mức phù hợp với người mới
- 25/100
Hướng nghiên cứu
Bắt đầu bằng cách đọc các đường dẫn định nghĩa và xử lý tùy chọn của mixlib-cli, sau đó xem xét nhu cầu được mô tả trong issue 38 và các ví dụ --connection-port, --winrm-session-timeout và --max-wait. Xác định phạm vi của việc xác thực số và xác thực phạm vi trước khi triển khai. Hoàn thành có nghĩa là các tùy chọn dòng lệnh có thể khai báo tiêu chí xác thực và nhất quán từ chối các giá trị không hợp lệ.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Describe the Enhancement:
At present, there is no way to add validation criteria while defining the options params for example if the option only accepts integer we need to validate by providing some option params.
It will also cover https://github.com/chef/mixlib-cli/issues/38
Describe the Need:
In https://github.com/chef/chef/pull/8489 we realized we need to pass an integer to Train, but we realized we should probably ensure the users are giving us an integer when we require that.
I don't see an existing pattern in Chef right now. Maybe it's a simple single line or maybe it's a pattern we make a mixin. Let's add validation to each of the command line options that should be numeric though.
For example, --connection-port, --winrm-session-timeout, --max-wait.
Maybe a mixlib-cli option that would let you specify that it should be numeric, and maybe the acceptable range? I kind of thought we already had something like that.
- Ngôn ngữ chính
- Ruby
- Star
- 118
- Fork
- 40
- 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
- Không có Dockerfile hay tệp Docker Compose
- Không có mẫu pull request
- Đọc 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 chef/mixlib-cli
-
Triage: Feature Request
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 35/100
chef/mixlib-cli#69 ·
-
Long options that start with `--no-*` are negated even when its explicitly defined as an optionĐang mởType: Bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 45/100
chef/mixlib-cli#48 · 1 bình luận ·
-
Option validationsĐang mởTriage: Feature Request
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 25/100
chef/mixlib-cli#38 · 1 bình luận ·
-
Triage: Feature Request
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 20/100
chef/mixlib-cli#27 ·
Tất cả issue của chef/mixlib-cli
Issue tương tự
-
bug
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
Maintainer thường phản hồi trong vòng 1 ngày
-
bug f:search
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
Maintainer thường phản hồi trong vòng 1 ngày
-
feature
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 60/100
manyfold3d/manyfold#7346 · 1 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
🐞 bug pending triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
CircuitVerse/CircuitVerse#8025 · 2 bình luận ·
Maintainer thường phản hồi trong vòng 1 ngày
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
we-promise/sure#4199 ·
Maintainer thường phản hồi trong vòng 1 ngày