Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Closed
wants to merge 2 commits into from

Conversation

Heart-Beats
Copy link

@Heart-Beats Heart-Beats commented Apr 19, 2024

#1306 修复 Gradle 版本过高相关方法删除导致的插件打包任务添加错误
#1049 修复插件中 context.getDatabasePath(name) 方法当 name 传入全路径时运行异常的问题

@tencent-adm
Copy link
Member

tencent-adm commented Apr 20, 2024

CLA assistant check
All committers have signed the CLA.

…本上添加失败的问题

CreatePackagePluginTask 类中相关被标记为废弃的方法进行替换,保证在后续 Gradle 版本的兼容性
…时运行异常的问题

对于这种携带路径的 name, makeSubName 方法返回原始值不添加前缀
@Heart-Beats Heart-Beats changed the title 修复 createPackagePluginTask 任务在 Gradle 7.5 以后版本上添加失败的问题 修复 createPackagePluginTask 任务在 Gradle 7.5 以后版本上添加失败和插件中 context.getDatabasePath(name) 方法当 name 传入全路径时运行异常的问题的问题 Apr 21, 2024
@shifujun
Copy link
Collaborator

shifujun commented Apr 22, 2024

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

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

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

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

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

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

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

@shifujun shifujun closed this Apr 22, 2024
@Heart-Beats
Copy link
Author

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

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

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

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

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants