Tencent / Shadow

零反射全动态Android插件框架
BSD 3-Clause "New" or "Revised" License
7.36k stars 1.29k forks source link

修复 createPackagePluginTask 任务在 Gradle 7.5 以后版本上添加失败和插件中 context.getDatabasePath(name) 方法当 name 传入全路径时运行异常的问题的问题 #1307

Closed Heart-Beats closed 2 months ago

Heart-Beats commented 2 months ago

1306 修复 Gradle 版本过高相关方法删除导致的插件打包任务添加错误

1049 修复插件中 context.getDatabasePath(name) 方法当 name 传入全路径时运行异常的问题

tencent-adm commented 2 months ago

CLA assistant check
All committers have signed the CLA.

shifujun commented 2 months ago

Hi,非常感谢和欢迎你提PR帮助项目改进,为此你肯定是花了不少时间的。

尽管Shadow项目并不活跃,但我们还是要尽可能的有一定的合入准则。 这些准则可能没有那么具体,但应该是大部分开源项目广泛的共识。

一般来说我会帮助提PR的人改进PR,尽可能以提出人的名义合入。 只是这个PR问题确实有点多。要解决的问题也不复杂,我就另外自己修复提PR了。 但是你的PR也是非常有价值的,我可以通过这个代码很清楚的看明白要解决的问题。

如果你下次还愿意提PR呢,希望能接受这些意见:

  1. 在修复问题时,尽可能少的修改代码。的确有很多代码不够优雅,顺手改了很方便。但这对于Code Review来说非常不便, 无法快速准确看出改动的影响。这不光是影响我们当前review PR是否能合入,更重要的是影响未来回溯问题时查看历史记录。特别是对于shadow这种不活跃的项目,历史记录尤为重要。对于代码的优雅性或其他改进可以另外提交。

  2. 一个PR应该围绕一个主题。你的两个提交几乎完全不相关。不能假设两个提交都是正确的,一起合入就没有问题。PR本身也是回溯问题的关键记录。

那么我先关闭这个PR了。88d252bba88f20f17c85e2da96b4a5c3997bfcff 我一会儿再测一下,没问题的话就帮你重新提个PR。

Heart-Beats commented 2 months ago

Hi,非常感谢和欢迎你提PR帮助项目改进,为此你肯定是花了不少时间的。

尽管Shadow项目并不活跃,但我们还是要尽可能的有一定的合入准则。 这些准则可能没有那么具体,但应该是大部分开源项目广泛的共识。

一般来说我会帮助提PR的人改进PR,尽可能以提出人的名义合入。 只是这个PR问题确实有点多。要解决的问题也不复杂,我就另外自己修复提PR了。 但是你的PR也是非常有价值的,我可以通过这个代码很清楚的看明白要解决的问题。

如果你下次还愿意提PR呢,希望能接受这些意见:

  1. 在修复问题时,尽可能少的修改代码。的确有很多代码不够优雅,顺手改了很方便。但这对于Code Review来说非常不便, 无法快速准确看出改动的影响。这不光是影响我们当前review PR是否能合入,更重要的是影响未来回溯问题时查看历史记录。特别是对于shadow这种不活跃的项目,历史记录尤为重要。对于代码的优雅性或其他改进可以另外提交。
  2. 一个PR应该围绕一个主题。你的两个提交几乎完全不相关。不能假设两个提交都是正确的,一起合入就没有问题。PR本身也是回溯问题的关键记录。

那么我先关闭这个PR了。88d252bba88f20f17c85e2da96b4a5c3997bfcff 我一会儿再测一下,没问题的话就帮你重新提个PR。

好的,没有问题,我这边是刚好遇到这些问题然后在自己能力范围之内给解决了,顺便提交了 PR, 因为这样会更方便帮助您定位解决问题,我这边还是更希望这些问题由您解决,因为 Shadow 本身项目还是具有挺高的复杂性,即使我本地验证通过可能还是存在相关问题,您这边验证没问题解决是再好不过的了。