OpenAtomFoundation / xupercore

The kernel of xuperchain.
Apache License 2.0
61 stars 47 forks source link

Support go lint #395

Open zhugelianglongming opened 1 year ago

zhugelianglongming commented 1 year ago

What changes

378

How tested

godeamon commented 1 year ago

这个 pr 改的代码也太多了

zhugelianglongming commented 1 year ago

这个 pr 改的代码也太多了

是的,但是没办法,开启报错之后,整个代码库都是错误。。。问题太多了。

代码库质量提升、集成自动校验肯定要经历这么一个过程,现在不改的话,以后也要改。除非要一直保留这些低质量代码。

这个代码库我改了非常久。

zhugelianglongming commented 1 year ago

@godeamon 有一些错误的处理,我把握不太好,辛苦你 reivew 下。

这个代码库中有太多的错误选择性忽略,不显示处理的话,太危险了。

godeamon commented 1 year ago

如果一定要改的话,我建议你按照模块来慢慢改,一个是方便 review,另一个即使出问题了也好回滚。

zhugelianglongming commented 1 year ago

如果一定要改的话,我建议你按照模块来慢慢改,一个是方便 review,另一个即使出问题了也好回滚。

这个建议好,我看看有没有拆分的办法。 拆分 PR:

zhugelianglongming commented 1 year ago

@godeamon 话说之前部分 patch 操作处理 error,部分不处理有什么讲究或者考虑在里面吗?