hotfix 时如果知道问题的位置,可以直接把这部分代码注释掉吗?

2020-03-22 17:18:29 +08:00
 1oNflow

hotfix 或者出 bug 的的标准操作是怎样的?

需要 revert 最近的提交,还是直接做出新的修改,比如直接修复,或者先注释掉新的提交(不影响核心功能),后面慢慢改?

899 次点击
所在节点    问与答
5 条回复
123444a
2020-03-22 18:22:07 +08:00
注释代码能解决问题?你的意思是取消某个功能吧?一般没有人会注释代码,都是用 flag 来控制功能的启动,就是 --help 出来那些 flag
yhxx
2020-03-22 19:22:10 +08:00
你说的是线上故障?
标准流程应该是先回滚,然后看情况再好好修
1oNflow
2020-03-22 21:22:51 +08:00
@yhxx
感谢回复,目前的情况是个 web 应用的前端部分( react ),有个 component 获取数据后显示的逻辑出错了,因为能定位到它,所以想直接注释掉这个 component 用户就看不见了,可以后续慢慢修。

回滚应该更好,但是需要回滚的部分不是最新 commit 的,中间也有别人开发的部分上线,害怕回滚了某个 commit 会影响别人的代码…

@123444a
目前用的是 nodejs,前端也并没有设定特定功能的 flag,这种做法在 java 后端用的比较多?
123444a
2020-03-22 21:25:37 +08:00
JS 开了混淆没有,开了就注释然后加个 ToDo,没开就删代码
1oNflow
2020-03-22 22:15:58 +08:00
@123444a 没,就写在 jsx 里的,小公司的项目没事,主要担心大公司流程是不是不能随便删(注释)代码就来问一下

这是一个专为移动设备优化的页面(即为了让你能够在 Google 搜索结果里秒开这个页面),如果你希望参与 V2EX 社区的讨论,你可以继续到 V2EX 上打开本讨论主题的完整版本。

https://www.v2ex.com/t/655129

V2EX 是创意工作者们的社区,是一个分享自己正在做的有趣事物、交流想法,可以遇见新朋友甚至新机会的地方。

V2EX is a community of developers, designers and creative people.

© 2021 V2EX