ZianTT / BHYG

B站 会员购 抢票 脚本
623 stars 107 forks source link

[重构]重构后的代码 #37

Closed bilibaiWater closed 1 year ago

bilibaiWater commented 1 year ago

还有一些交互逻辑上的优化&更改

bilibaiWater commented 1 year ago

+845-500的pr(笑 能过吗xD

ZianTT commented 1 year ago

啊?好可怕…等我恰完饭马上review

ZianTT commented 1 year ago

妈呀有点恐怖readme都改了,我单独开个分支吧,我先把我的不分并入main,你的我慢慢并…害 对了别删sentry啊我好跟踪…

ZianTT commented 1 year ago

抱歉更改太多可能不能直接合并…我单独开个分支记录你的更改吧(感觉可以当一个major version用了)

bilibaiWater commented 1 year ago

我把那个云log库(没用过)删了,改成本地logging,因为打包出来太大了。不加7mb,加了50mb,所以...

另,这个pr肯定不能直接merge到main里啊(笑)不然都快成我的仓库了(笑x2)

我对“木兰许可证”并不熟,不敢直接扒下来发自己仓库xD

所以文件头的许可我这么写应该是对的吧..?

ZianTT commented 1 year ago

我把那个云log库(没用过)删了,改成本地logging,因为打包出来太大了。不加7mb,加了50mb,所以...

另,这个pr肯定不能直接merge到main里啊(笑)不然都快成我的仓库了(笑x2)

我对“木兰许可证”并不熟,不敢直接扒下来发自己仓库xD

~所以文件头的许可我这么写应该是对的吧..?~

其实事实上已经违背了,文件头理论上你不能改,只能在README中写明你修改了,并且要标明原作出处

bilibaiWater commented 1 year ago

all right...看来我对于开源这方面还需要学习...

好吧...那我应该删掉这个fork,重新开一个?然后发pr?还是..?

ZianTT commented 1 year ago

all right...看来我对于开源这方面还需要学习...

好吧...那我应该删掉这个fork,重新开一个?然后发pr?还是..?

pr我关掉了,fork你可以直接进行修改(其实也没多大事