3
我的卧龙凤雏队友
source link: https://www.v2ex.com/t/935569
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
多次强制 Push 代码到主分支，我的提交过的代码被覆盖过一次
项目循环导包，golang 项目竟然出现了循环导包, internal 里面对外导出
"令人震惊"的接口设计
修改密码时校验验证码竟然是 api 接口，关键前端还接了
18 条回复 • 2023-04-26 13:51:58 +08:00
 |
0littleboy 2 小时 51 分钟前 讲清楚就好了，你就应该让他 PR
|
 |
yolooo 2 小时 48 分钟前 主分支设 protected 啊，说明合并需求走 MR 啊
|
 |
tanghanyu 2 小时 45 分钟前 主分支至少加个保护吧，我们还会设置主分支 push 强制触发评审？
|
 |
lovedebug 2 小时 45 分钟前 收回权限，branch 设置 rule ，不允许强制 push ，只有 Admin 可以。
开发只能在 feat branch 上，只能 PR 或者 merge 到主分支 |
 |
coderluan 2 小时 44 分钟前 如果你的同事是卧龙凤雏，那么只能说明你们的领导是阿斗。
|
1 |
kera0a 2 小时 38 分钟前 via iPhone 但说不定你同事这么屌的代码用到公司倒闭也不会有什么问题
|
 |
iyiluo 2 小时 38 分钟前 主干加权限，有修改提交申请合并
|
 |
kaokao123 2 小时 35 分钟前 |
 |
optional 2 小时 18 分钟前 via iPhone 这是同事问题吗？这是领导管理问题。
|
 |
Glauben 2 小时 14 分钟前 看这个接口设计，确实是同事问题啊？这技术，这态度。上面的权限设计遇到这种队友确实很有必要
|
 |
SuperManNoPain 2 小时 12 分钟前 有点吓人了 哈哈哈哈
|
 |
qvsdghsas 2 小时 10 分钟前 说明你们公司挺二的，说一声就得了 告诉他以后不要强制 push 到主分支了 主分支设置一下保护
|
 |
luobogooooo 2 小时 4 分钟前 「多次强制 Push 代码到主分支」这种事情我有一家公司的老板经常干😂
|
 |
xuelu520 2 小时 2 分钟前 主分支锁了就行，合并走 PR ，这样还能稍微 review 下代码
|
 |
gitignore 1 小时 56 分钟前 「修改密码时校验验证码竟然是 api 接口，关键前端还接了」
有什么问题吗，指的是「不应该用 GET 请求方法」 还是「通过接口校验验证码」而非「将用户输入」 post 到服务端校验。 「通过接口校验成功返回一个 token ，前端再将 token 及登陆信息 post 到登陆接口」这个机制好像在一些平台也见过吧，特别是用了第三方验证码 sdk 的，好像抖店就是？ |
 |
darkengine 1 小时 26 分钟前 @gitignore 如果是校验通过拿到 token ，修改密码的时候把这个 token 带上那确实没问题。就怕是前端用校验接口校验通过后，发送修改密码请求的时候不带 code 和 token 了。
|
 |
artnowben 4 分钟前 对新人做好培训
|
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK