add config validation by default
还没有人认领这个 Issue。
评估
- 难度
- 4/5
- 预计耗时
- 3-5 天
- 新手友好度
- 45/100
- Issue 类型
- 功能
- 描述清晰度
- 描述清楚
- 活跃度
- 停滞
- 技术栈
- javascript
- 领域
- cli
调研方向
从 src/BaseCommand.js 开始，重点查看 getFullConfig 和现有的 flag 处理，然后结合仓库中对 getFullConfig 的搜索结果检查 src/commands/app/pack.js 和 src/commands/app/deploy.js。确保继承的命令 spread BaseCommand 的 flags，并确保配置验证默认开启但可以禁用；验证受影响命令的行为和现有测试。
由索引模型根据 Issue 内容生成。
描述
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
- 主要语言
- JavaScript
- 星标
- 28
- 派生
- 41
- PR 合并指标
- 30 天内没有已合并 PR
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
adobe/aio-cli-plugin-app 的其他 Issue
-
app:pack aborts when Node writes process warnings to api-mesh stderr可能已有人在做 @raj-prince587 于 87 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
adobe/aio-cli-plugin-app#930 ·
-
bug
难度 3/5 1-2 天 新手友好度 48/100
adobe/aio-cli-plugin-app#924 · 1 条评论 ·
-
难度 3/5 1-2 天 新手友好度 48/100
adobe/aio-cli-plugin-app#919 ·
-
难度 5/5 一周以上 新手友好度 35/100
adobe/aio-cli-plugin-app#890 ·
-
bug
难度 4/5 3-5 天 新手友好度 28/100
adobe/aio-cli-plugin-app#881 · 1 条评论 ·
查看 adobe/aio-cli-plugin-app 的全部 Issue
相似的 Issue
-
neutral:fix
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
wazuh/wazuh-documentation#10611 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 62/100
remotion-dev/remotion#12905 ·
维护者通常 1 天内回复
-
enhancement websites
难度 2/5 1-3 小时 新手友好度 78/100
quarto-dev/quarto-cli#15255 · 1 条评论 ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
PolicyEngine/cliff-watch#44 ·