Replace repeated `x.get(y) or {}` chaining with a safer accessor
还没有人认领这个 Issue。
评估
调研方向
首先阅读 services/hackbot-pulse-listener/app/{taskcluster,github,notify}.py 和 services/hackbot-api/app/{actions_applier,routers/events,phabricator_webhook,routers/webhooks}.py 中重复的访问模式。比较共享 accessor 或在 parsing 边界进行规范化哪一种更适合现有流程。完成的标准是,在不改变可选字段行为的情况下,统一替换重复的防御性 fallback。
由索引模型根据 Issue 内容生成。
描述
This defensive-defaulting idiom is widespread, for example in services/hackbot-pulse-listener/app/{taskcluster,github,notify}.py and services/hackbot-api/app/{actions_applier,routers/events,phabricator_webhook,routers/webhooks}.py. Introduce a small helper, or normalize optional fields at the parsing boundary, instead of re-deriving {} fallbacks ad hoc throughout the codebase.
- 主要语言
- Python
- 星标
- 570
- 派生
- 351
- 平均合并
- 1 天 19 小时
- 30 天内合并 PR
- 66
贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
mozilla/bugbug 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 70/100
-
good-first-bug hackbot
难度 2/5 1-3 小时 新手友好度 65/100
-
good-first-bug hackbot
难度 1/5 1-3 小时 新手友好度 88/100
-
难度 2/5 1-3 小时 新手友好度 78/100
-
hackbot
难度 2/5 1-3 小时 新手友好度 82/100
相似的 Issue
-
bug
难度 2/5 1-3 小时 新手友好度 75/100
stephrobert/dsoxlab#238 ·
-
难度 2/5 1-3 小时 新手友好度 75/100
-
难度 2/5 1-3 小时 新手友好度 75/100
sublimehq/package_control#1780 ·
-
难度 2/5 1-3 小时 新手友好度 65/100
-
难度 2/5 1-3 小时 新手友好度 70/100
nwg-piotr/nwg-displays#145 ·