万人血书把控P开头老哥PR质量

[Meo597]

老哥全靠臆想修bug，全靠臆想新增功能
他甚至都没有自己本地测一下是否行得通

比如dns功能被合并进去几个月，我发现他自己都不用，也根本用不了

比如今年初他以滥用cfworker的目的，非要在route新增字段，对它大幅度修改
我个人认为不靠谱，明明有改动极小、更直接的解决方式

友好交涉几天无果，他决定动dns，新增unexceptIPs，本以为事情告一段落了
然而，半年过去了，前几天他又动freedom，理由是为了滥用cfworker，我提醒他你半年前不是已经用dns实现目的吗？他沉默不语

这实锤了他靠臆想新增功能，未经验证，甚至自己都不用
现在又搞出内存泄漏问题

PR撕逼是一把好手，他有无限的经历和时间花在上面

我现在高度怀疑他代码是ai写的，他自己根本不懂，甚至不是进阶用户，因为正常的开发者不会用奇怪的思路去实现需求

为了把控core代码质量，强烈建议新开dev子项目，所有非核心开发者的pr，全部进dev，稳定几个月后再合并到core

[wyx2685]

@Meo597 新晋AI编程领域大神了属于是

[Meo597]

@Meo597 新晋AI编程领域大神了属于是

我高度怀疑他用ai逐行询问有没有”bug”

因为正常的码农不会用这些奇怪的思路，绕了一大圈实现需求

用都用不好，怎么能提pr呢

[patterniha]

almost all my new features are disabled by defaults, and all my found-bugs-PRs are closed by @Fangliding and then rewritten by him.

@RPRX I hope you confront against this blatant racism.

[Meo597]

一边喊着种族歧视，一边玩滥权
嘴上一套，行为一套

玩得很熟练，味儿太浓了

装都不装了直接关issue
你不避嫌的吗？

[xqzr]

这...的确不是 issue😇
> https://github.com/XTLS/Xray-core/discussions

[simpleandstupid]

https://xtls.github.io/development/intro/guide.html

[patterniha]

我高度怀疑他用ai逐行询问有没有”bug”

Just look at this and see who used AI.

I have no interest in continuing this pointless discussion. I am looking for the main problem.

[Meo597]

1. 我golang菜我自己知道，这不是我熟悉的语言，所以我不会像你一样，疯狂给core提pr。菜不自知最可怕
2. 这个pr只是概念验证，注释ai写的，代码手写的。而且确实解决了某些需求
3. 你玩滥权关issues是心虚了吗？

[Meo597]

滥用你的权力捂嘴，然后扣着种族歧视的大帽子
这味道太熟悉了

[Meo597]

你为什么要关闭issues？唯一理由和动机只能是不想让人看到

因为你每天给core提交屎山代码，浪费了我许多时间维护自己的分支，我相信不止我一个人这样做

不要再制造垃圾了

[Meo597]

你为什么不敢正面回应：
你为了滥用cf worker魔改的dns

当时你信誓旦旦的说能满足需求
时隔半年，为什么又因此魔改freedom

回答不出来，就是你菜，完全依赖ai写自己都看不懂的代码

[RPRX]

1. 关于新功能的 PR，如果不影响现有代码（默认关闭）一般可以接受，不然没有新鲜血液了，反正后面没人了用的话可以删掉
2. 关于较新的贡献者修复 bug，以后应当遵循 Revert "Chore: Three small 'fixes'" #5057 (comment)
3. 这里没有种族歧视，但 issue 也确实不是该发这样内容的地方，如果哪个 commit 出问题了可以发具体的 issue

[Meo597]

这老哥有巨多精力在pr撕逼
我们没有时间精力
去审计他的每一行代码
用人的精力对抗他的ai额度，怎么可能赢？

ray的屎山代码对开发者要求更高
如果只是加开关堆新功能其实无所谓
大家应该都没什么意见

这老哥太喜欢重构、修没有验证的bug

导致压力全给到 @Fangliding
现在他还能自己reopen，Fangliding也快受不了了估计

针对非核心开发者，应该更审慎的处理这种pr
至少应该放到alpha版本几个月

[RPRX]

其实主要是因为 *ray 的代码架构太屎了，各种没注释的注入、ctx 搞得太多，很多时候看似一处修复却可能会导致其它地方出 bug

不是说不能这么设计，而是注释不全的话只有自己记得、别人改代码很容易搞出 bug，甚至自己几个月后都会忘，还是 vr 的锅

弄完 VLESS 加密后我先把各处没必要的 interface 给清掉吧，vr 写的代码不管干啥都先要来个 interface、register，也是服了