jitwxs / 163MusicLyrics

Windows 云音乐歌词获取【网易云、QQ音乐】
Apache License 2.0
2.08k stars 108 forks source link

重构代码, 修复Bug #53

Closed textGamex closed 2 years ago

textGamex commented 2 years ago

变动

textGamex commented 2 years ago

提个建议, 你下次可以把函数写短点吗? 多拆出几个函数, 函数太长看的好累:confounded:

textGamex commented 2 years ago

这次更新的时候, 更新日记能把贡献者的名字写上去吗? 就像这样

Fixed Bugs

jitwxs commented 2 years ago

这次更新的时候, 更新日记能把贡献者的名字写上去吗? 就像这样

Fixed Bugs

  • xxxxx (贡献者: xxx)

可以的

jitwxs commented 2 years ago

提个建议, 你下次可以把函数写短点吗? 多拆出几个函数, 函数太长看的好累😖

哈哈,自己写比较随性了 :joy:

jitwxs commented 2 years ago

晚点我看看,MR 建议每一块改动提一个 MR,聚在一个里面 CR 看起来比较累

textGamex commented 2 years ago

晚上点我看看,MR 建议每块砖提一个 MR,聚在一个 CR 里面比较累

不懂就问, MR和CR是什么意思?

jitwxs commented 2 years ago

晚上点我看看,MR 建议每块砖提一个 MR,聚在一个 CR 里面比较累

不懂就问, MR和CR是什么意思?

MR(merge request) 是 gitlab 那边的词,对应 github 应该是 PR(pull request);CR 是 code review

textGamex commented 2 years ago

晚上点我看看,MR 建议每块砖提一个 MR,聚在一个 CR 里面比较累

不懂就问, MR和CR是什么意思?

MR(merge request) 是 gitlab 那边的词,对应 github 应该是 PR(pull request);CR 是 code review

哦哦, 原来如此, 谢谢

textGamex commented 2 years ago

提个建议, 你下次可以把函数写短点吗? 多拆出几个函数, 函数太长看的好累😖

哈哈,自己写比较随性了 😂

作者你能把你的下一步开发计划公布出来吗?我好规划我的下一步工作

jitwxs commented 2 years ago

提个建议, 你下次可以把函数写短点吗? 多拆出几个函数, 函数太长看的好累😖

哈哈,自己写比较随性了 😂

作者你能把你的下一步开发计划公布出来吗?我好规划我的下一步工作

目前的开发计划主要是视 issuse 来定,我建了一个 projects,没有 Assignees 的任务都可以提 PR。

textGamex commented 2 years ago

提个建议,你下次可以把函数写成短点吗?多出几个函数,函数太长看的好累😖

哈哈,自己写比较随性了😂

作者你把你的下一步开发计划发布出来了吗?

目前的开发计划主要是看问题来定,我建了一个项目,没有Assignees的任务都可以提PR。

OK