code shouldn't be stored as character
还没有人认领这个 Issue。
评估
调研方向
首先阅读 R/qenv-eval_code.R 中所引用的第 36 行和第 94-96 行，以追踪 quenv() |> within() 附近当前的字符转换。然后检查 teal.data 的 eval_code.teal_data_module 实现，以比较相关行为。当不必要的转换在两个位置都得到一致处理，且不改变代码评估行为时，即视为完成。
由索引模型根据 Issue 内容生成。
描述
In R, there are more appropriate structures to store the code but we keep storing this as a character.
This has some indirect negative consequences of unnecessary type conversions. Right now, if I call quenv() |> within({<code>}) then my <code> is converted to character and then back again to the code. Such behaviour is currently being adapted in teal.data (eval_code.teal_data_module) so if changed - please change there as well.
- 主要语言
- R
- 星标
- 12
- 派生
- 11
- 平均合并
- 8 天 15 小时
- 30 天内合并 PR
- 2
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
insightsengineering/teal.code 的其他 Issue
-
Move to Pharmaverse可能已有人在做 @llrs-roche 于 38 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 68/100
insightsengineering/teal.code#292 ·
-
[Feature Request]: High cyclomatic complexity leds to bad readability可能已有人在做 @llrs-roche 于 39 天前认领。 未关闭enhancement
难度 3/5 3-5 天 新手友好度 45/100
insightsengineering/teal.code#295 ·
-
core discussion
难度 5/5 一周以上 新手友好度 25/100
insightsengineering/teal.code#291 · 2 条评论 · 1 个 reaction ·
-
bug core
难度 3/5 1-2 天 新手友好度 45/100
insightsengineering/teal.code#275 ·
-
bug core
难度 4/5 3-5 天 新手友好度 38/100
insightsengineering/teal.code#272 ·
查看 insightsengineering/teal.code 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 64/100
mitchelloharawild/ggtime#39 ·
-
enhancement good first issue help wanted
难度 2/5 半天 新手友好度 82/100
-
难度 2/5 1-3 小时 新手友好度 62/100
-
question
难度 2/5 1-3 小时 新手友好度 62/100
-
Breaking 🏴☠️ Consistency :green_apple: :apple:
难度 1/5 1 小时以内 新手友好度 70/100
easystats/bayestestR#789 ·
维护者通常 1 天内回复