记一次给GORM提issues及PR被拒的经历

145 阅读1分钟

问题来源

GORM踩坑

提交playground

  • 这里是用来复现BUG的地方
  1. fork go-gorm/playground
  2. 更改测试, 确保能复现改问题
  3. 提交代码
  4. 提交PR
  5. PR示例

创建issues

  • 关联刚刚创建的playground PR, 使得我们的问题和复现方式形成关联, 以确保提交的issues是真实可复现的
  1. 创建issues
  2. 关联刚刚提交的playground
  3. issues示例

提交PR

  • 这一步比较繁琐, 除了你要去解决该issues, 还要保证能通过github action前置校验, 这里面包含了代码规范, 测试用例等, 通过这些检查后PR才算建立完成
  1. fork go-gorm/gorm
  2. 更改源文件
  3. 提交代码
  4. 创建PR
  5. PR示例

剩下的就是等待审核了......

回复来了

  • gorm.io/docs/query.…

  • When the type is string, we cannot actually confirm that the user wants to query through the primary key

db.First(&user, "1=1")

image.png

总结

  1. 思考还是太简单了, 这种简单的场景都没有考虑到
  2. 在尝试去解决这种类型的问题时候, 应该考虑更多的场景, 以保证我们能真正的解决问题