Skip to content

feat:将牌阵结果解析纳入转发列表 #359

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

Merged
merged 85 commits into from
Aug 4, 2022

Conversation

shudorcl
Copy link
Member

@shudorcl shudorcl commented Aug 3, 2022

将牌阵结果解析纳入转发列表

不过怎么commit列表还是那么长。。。

- fortune的设置底图功能
- b14的加密功能
- 加密
- 哔哩哔哩推送
- 藏头诗
- 运势
- 加上了因为复制粘贴疏忽又没有注意测试遗漏的`?`
- 调整藏头诗和加密的正则触发,使其不必多此一举
- 删去了未被发现的测试代码
- 优化了music bilibili image_finder 中使用空格的体验
- 补上了plugin_bilibili中未实现的vup开头触发
- 为plugin_bilibili_parse输出的消息加上一个换行符,优化排版
- 补回readme中bilibili_push的注释说明
- 逆位的图片现在可以倒过来了
@guohuiyuan
Copy link
Contributor

将牌阵结果解析纳入转发列表

不过怎么commit列表还是那么长。。。

最用master同步上游分支-----在自己其他分支做修改pr到上游, 自己的master再去同步上游的master

@shudorcl
Copy link
Member Author

shudorcl commented Aug 3, 2022

最用master同步上游分支-----在自己其他分支做修改pr到上游, 自己的master再去同步上游的master

问题正在于此,实际上这个pr一开始是打算从其他分支pr到上游的:
(在上次pr后,在GitHub网页同步上游时也选择了discard changes,然后再分支出去进行修改)
image
但是依然出现了先前的commit
于是想尝试一下合并到fork的master能否有所改观,可惜如现在所见并没有

@guohuiyuan
Copy link
Contributor

你下次,然后回滚自己的master,然后同步上游master,就不会有问题了,或者删掉重新fork

@fumiama
Copy link
Member

fumiama commented Aug 3, 2022

删掉重新fork最快了

@fumiama fumiama merged commit 0e29010 into FloatTech:master Aug 4, 2022
@shudorcl
Copy link
Member Author

shudorcl commented Aug 4, 2022

最用master同步上游分支-----在自己其他分支做修改pr到上游, 自己的master再去同步上游的master

问题正在于此,实际上这个pr一开始是打算从其他分支pr到上游的

重新测试了一下,问题似乎是在本地的master分支。。。

从本地master分支出去修改
image
从远程master分支出去修改
image

结论大概是。。。重新clone一个fork下来的仓库

没必要,只需要在本地master分支来一发git reset --hard {远程分支的哈希}就好了

image

@fumiama
Copy link
Member

fumiama commented Aug 4, 2022

你会高级的当然就好说了,重新fork最省事但确实不是最优解。最优解是添加一个新origin是上游,然后从上面拉取一个新分支,修改后PR。

@shudorcl
Copy link
Member Author

shudorcl commented Aug 4, 2022

好有道理,学到了

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.

4 participants