多次强制 Push 代码到主分支，我的提交过的代码被覆盖过一次
项目循环导包，golang 项目竟然出现了循环导包, internal 里面对外导出
"令人震惊"的接口设计
修改密码时校验验证码竟然是 api 接口，关键前端还接了
This topic created in 1138 days ago, the information mentioned may be changed or developed.
 |
1
0littleboy Apr 26, 2023
讲清楚就好了，你就应该让他 PR
|
 |
2
yolooo Apr 26, 2023
主分支设 protected 啊，说明合并需求走 MR 啊
|
 |
3
tanghanyu Apr 26, 2023
主分支至少加个保护吧，我们还会设置主分支 push 强制触发评审？
|
 |
5
lovedebug Apr 26, 2023
收回权限，branch 设置 rule ，不允许强制 push ，只有 Admin 可以。
开发只能在 feat branch 上，只能 PR 或者 merge 到主分支 |
 |
6
coderluan Apr 26, 2023  5
如果你的同事是卧龙凤雏，那么只能说明你们的领导是阿斗。
|
 |
7
kera0a Apr 26, 2023 via iPhone 1
但说不定你同事这么屌的代码用到公司倒闭也不会有什么问题
|
 |
8
iyiluo Apr 26, 2023
主干加权限，有修改提交申请合并
|
 |
9
kaokao123 Apr 26, 2023
用规则
|
 |
10
optional Apr 26, 2023 via iPhone
这是同事问题吗？这是领导管理问题。
|
 |
11
Glauben Apr 26, 2023
看这个接口设计，确实是同事问题啊？这技术，这态度。上面的权限设计遇到这种队友确实很有必要
|
 |
12
SuperManNoPain Apr 26, 2023
有点吓人了 哈哈哈哈
|
 |
13
qvsdghsas Apr 26, 2023
说明你们公司挺二的，说一声就得了 告诉他以后不要强制 push 到主分支了 主分支设置一下保护
|
 |
14
luobogooooo Apr 26, 2023
「多次强制 Push 代码到主分支」这种事情我有一家公司的老板经常干😂
|
 |
15
xuelu520 Apr 26, 2023
主分支锁了就行，合并走 PR ，这样还能稍微 review 下代码
|
 |
16
gitignore Apr 26, 2023
「修改密码时校验验证码竟然是 api 接口，关键前端还接了」
有什么问题吗，指的是「不应该用 GET 请求方法」 还是「通过接口校验验证码」而非「将用户输入」 post 到服务端校验。 「通过接口校验成功返回一个 token ，前端再将 token 及登陆信息 post 到登陆接口」这个机制好像在一些平台也见过吧，特别是用了第三方验证码 sdk 的，好像抖店就是？ |
 |
17
darkengine Apr 26, 2023
@gitignore 如果是校验通过拿到 token ，修改密码的时候把这个 token 带上那确实没问题。就怕是前端用校验接口校验通过后，发送修改密码请求的时候不带 code 和 token 了。
|
 |
18
artnowben Apr 26, 2023
对新人做好培训
|
 |
19
me404 Apr 26, 2023
主分支不保护？
|
 |
20
fiypig Apr 26, 2023
难道是校验验证码验证过，直接修改密码不校验吗？
|
 |
21
zhiouzhou Apr 26, 2023
所以卧龙的队友是凤雏，感觉很合理啊（🤪🐶
|
 |
22
zhandi4 Apr 26, 2023
第一个问题就不止你队友是卧龙了
|
 |
24
chenPiMeiHaoChi Apr 26, 2023 4
但这些人就是能找到工作，工资还不低。我也不懂为啥，现在看开了，都是无产阶级，能糊就糊一块糊吧，这种篓子说一声下回别再这样就行了。
|
 |
25
DeepRedApple Apr 26, 2023 via iPhone
那有啥，我之前某位同事发布上线项目竟然是直接更新.class 文件，现在工作的数据库连索引都不加，该加不加那种。
|
 |
26
qiaobeier Apr 26, 2023
同款队友，还是个漂亮妹子，我都是默默的 git reflog 🤣
|
 |
27
chengxiao Apr 26, 2023
说不定人家工资比你还高呢 ~
|
 |
28
abersheeran Apr 26, 2023
|
 |
30
littlewing Apr 26, 2023
你再强制给他 push 覆盖回去
|
 |
31
huzhizhao Apr 27, 2023
你跟代码有一个能跑就行
|
 |
34
JKeita Apr 27, 2023
主分支不设权限？
|
Select Language