Change `symbolize_names` to `symbolize_keys`
还没有人认领这个 Issue。
评估
调研方向
先在 repository 中搜索 symbolize_names,包括其文档和测试,然后检查该选项是如何被解析和公开的。确认 symbolize_keys 是否已在其他地方使用,并定义兼容性范围;当约定的名称得到一致处理,且相关测试和文档均通过时,即视为完成。
由索引模型根据 Issue 内容生成。
描述
This occured to me while testing this option without the docs in front of me; it didn't work because I thought the option was named symbolize_keys (and only one day before I wrote the docs for it...). Why symbolize_names? symbolize_keys describes exactly what it does, symbolizing Hash keys, not names. Also, symbolize_keys already has some spread as a method name.
I will provide a PR if there is agreement on this.
- 主要语言
- Ruby
- 星标
- 596
- 派生
- 223
- 平均合并
- 8 小时 38 分钟
- 30 天内合并 PR
- 2
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
ruby/psych 的其他 Issue
-
难度 2/5 1-3 小时 新手友好度 78/100
-
Documentation
难度 1/5 1 小时以内 新手友好度 72/100
-
Psych.dump raises Encoding::CompatibilityError for UTF-16/UTF-32 strings可能已有人在做 @Halvanhelv 于 54 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 55/100
-
难度 4/5 3-5 天 新手友好度 68/100
-
难度 4/5 3-5 天 新手友好度 52/100
相似的 Issue
-
JWT::EncodedToken raises TypeError for a non-object header可能已有人在做 @anakinj 今天认领。 未关闭bug
难度 2/5 1-3 小时 新手友好度 78/100
jwt/ruby-jwt#780 · 1 个 reaction ·
维护者通常 1 天内回复
-
难度 1/5 1 小时以内 新手友好度 82/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 78/100
-
难度 2/5 1-3 小时 新手友好度 82/100
zold-io/zold-ruby-sdk#85 ·
-
难度 1/5 1-3 小时 新手友好度 62/100