gaogaotiantian / biliscope

Bilibili chrome extension to show uploader's stats
MIT License
575 stars 45 forks source link

feat: Support sync with gitee in option #43

Closed CaiJingLong closed 1 year ago

Dynesshely commented 1 year ago

没有理解为什么要增加上传到 gitee 的功能, 请在描述中说明

另外, 建议先开 issue 讨论功能的必要性, 确认要做再提交 PR

gaogaotiantian commented 1 year ago

首先暂时不会增加这个功能,它有挺多的问题,比如要求你的gitee的repo是某个特别的格式。另外access token这种形式感觉也是有点小问题,适用性太窄了。另外通过这种形式去和其他网站的API进行通讯会让这个插件的审批更麻烦一些。

另外,当你改代码的时候,你得尊重原来代码的格式……不能把原来的代码全给调成你觉得正确的状态,你把我缩进全都改了,这样的PR是肯定不会被接受的。

最后就是,确实在增加这种相对比较大的feature之前应该先提一个issue,因为这不是简单的bug fix。目前来说,还没有打算增加这个和gitee同步的feature。

CaiJingLong commented 1 year ago

你的仓库你说了算 同步算是一个比较常见的需求,提供一个手段罢了,不容易上商店这个理由我是可以接受的的 close 掉无所谓。 Feat这东西,我可以提,你可以不接受,但是以修改了代码格式化方案不同为理由来说事是很可笑的,你原代码只有100行,我新增代码200行,加上对原代码有重构,使用正常的格式化工具无可厚非,我查阅了仓库,未发现developer.md这样的文档有任何要求,你可以要求修改未更改部分的代码为原格式,或提供你认为正确的格式化工具或文档格式,代码review的目的就在于此。

另外,你说先开 issue询问是否有必要,你觉得是对你的不尊重,所以直接关pr? 直接close对我来说也不是太尊重,你不觉得代码我已经写了,应该先商量一下?或者说明原因等我回复?简单来说,尊重是相互的。我在你这里没有看到尊重,所以可以不谈尊重本身。

Dynesshely commented 1 year ago

你的仓库你说了算 同步算是一个比较常见的需求,提供一个手段罢了,不容易上商店这个理由我是可以接受的的 close 掉无所谓。 Feat这东西,我可以提,你可以不接受,但是以修改了代码格式化方案不同为理由来说事是很可笑的,你原代码只有100行,我新增代码200行,加上对原代码有重构,使用正常的格式化工具无可厚非,我查阅了仓库,未发现developer.md这样的文档有任何要求,你可以要求修改未更改部分的代码为原格式,或提供你认为正确的格式化工具或文档格式,代码review的目的就在于此。

另外,你说先开 issue询问是否有必要,你觉得是对你的不尊重,所以直接关pr? 直接close对我来说也不是太尊重,你不觉得代码我已经写了,应该先商量一下?或者说明原因等我回复?简单来说,尊重是相互的。我在你这里没有看到尊重,所以可以不谈尊重本身。

1) 你事先未使用 issue 与仓库所有者沟通协商, 且 pr 内容并非 bug fix 2) 你的 PR 没有任何描述, 没人有功夫直接读你的 commit 3) @gaogaotiantian 将 close pr 的理由解释得非常清楚, 我不认为没有不妥当的言辞 4) 提交代码需要尊重原本的代码样式 5) 社区对使用 gitee 同步的功能并没有很大的呼声, 属于非必要 feat, 你应当使用协商与建议的态度 6) 严格来说, 你新增代码应当是一个 commit, 重构源代码应当是另外一个 commit 7) 在开源社区, 直接 close pr 并非不尊重的行为, 况且 @gaogaotiantian 并非没有做出解释 8) 你代码已经写了就应该与你商量吗 ? 你也并没有开 issue 来询问 @gaogaotiantian 该功能是否有可能被合并 9) 在你先前并未尊重作者的情况下, 你也没有资格谈尊重, 何况 @gaogaotiantian 谈的尊重针对代码而非人

以上