add config validation by default
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 45/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Đặc tả rõ ràng
- Mức độ hoạt động
- Đình trệ
- Công nghệ
- javascript
- Lĩnh vực
- cli
Hướng nghiên cứu
Bắt đầu với src/BaseCommand.js, đặc biệt là getFullConfig và cách xử lý flag hiện có, sau đó kiểm tra src/commands/app/pack.js và src/commands/app/deploy.js cùng với các kết quả tìm kiếm trong repository cho getFullConfig. Đảm bảo các lệnh kế thừa spread các flag của BaseCommand và việc xác thực cấu hình được bật theo mặc định nhưng có thể bị tắt; kiểm tra hành vi của các lệnh bị ảnh hưởng và các test hiện có.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Right now, config validation is not turned on by default and only certain commands turn it on, like the app pack command:
https://github.com/adobe/aio-cli-plugin-app/blob/4fb2a25f56050e31d7f444b80c6261ca609d688e/src/commands/app/pack.js#L43
Modify BaseCommand.getFullConfig to validate the app config by default - this means that all commands that call getFullConfig will validate the config by default, even if they don't use the whole config:
https://github.com/search?q=repo%3Aadobe%2Faio-cli-plugin-app%20getFullConfig&type=code
BaseCommand.getFullConfig:
https://github.com/adobe/aio-cli-plugin-app/blob/4fb2a25f56050e31d7f444b80c6261ca609d688e/src/BaseCommand.js#L147
Provide a shared flag in the BaseCommand to turn off config validation, just in case there are issues:
- The flag should be called
config-validationandallowsNo : true,default: true. - The
getFullConfigmethod should read the flag directly - every command that inherits from BaseCommand must spread the
BaseCommandflags
- Ngôn ngữ chính
- JavaScript
- Star
- 28
- Fork
- 41
- Merge trung bình
- 5 giờ 54 phút
- Pull request đã merge (30 ngày)
- 4
Chuẩn bị môi trường
- Không có Dockerfile hay tệp Docker Compose
- 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 adobe/aio-cli-plugin-app
-
app:pack aborts when Node writes process warnings to api-mesh stderrCó thể đã có người làm @raj-prince587 đã nhận 79 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
adobe/aio-cli-plugin-app#930 ·
-
bug
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
adobe/aio-cli-plugin-app#924 · 1 bình luận ·
-
Độ khó 3/5 1-2 ngày Mức phù hợp với người mới 48/100
adobe/aio-cli-plugin-app#919 ·
-
Độ khó 5/5 Hơn một tuần Mức phù hợp với người mới 35/100
adobe/aio-cli-plugin-app#890 ·
-
bug
Độ khó 4/5 3-5 ngày Mức phù hợp với người mới 28/100
adobe/aio-cli-plugin-app#881 · 1 bình luận ·
Tất cả issue của adobe/aio-cli-plugin-app
Issue tương tự
-
bug require-triage
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 72/100
expressjs/expressjs.com#2549 · 1 reaction ·
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 82/100
-
Độ khó 1/5 Dưới một giờ Mức phù hợp với người mới 85/100
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
vercel-labs/agent-skills#348 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 65/100
NanmiCoder/dsh-agent-teams#289 ·
Maintainer thường phản hồi trong vòng 1 ngày