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

修改梦见月瑞希的名字缩写,抽卡统计里已经换行了 #892

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

1055169799
Copy link
Contributor

对了,抽卡统计/武器统计,这种什么时候做一下分栏,图片已经很长了喵

@jkl8753
Copy link
Contributor

jkl8753 commented Feb 17, 2025

有个小建议,像890的添加蓝砚英文名,和889的修改集录时间这些等,
这些都是小改动,完全可以合在同一个PR里面一起提交。
还有你891 892两个都是改梦见月昵称?改动内容都一模一样,考虑关掉其中一个PR?

@1055169799
Copy link
Contributor Author

有个小建议,像890的添加蓝砚英文名,和889的修改集录时间这些等,
这些都是小改动,完全可以合在同一个PR里面一起提交。
还有你891 892两个都是改梦见月昵称?改动内容都一模一样,考虑关掉其中一个PR?

呃…我不知道怎么把多个改动合并在一块儿提交,两个昵称改动,改动的是不同位置的文件

@jkl8753
Copy link
Contributor

jkl8753 commented Feb 17, 2025

有个小建议,像890的添加蓝砚英文名,和889的修改集录时间这些等,
这些都是小改动,完全可以合在同一个PR里面一起提交。
还有你891 892两个都是改梦见月昵称?改动内容都一模一样,考虑关掉其中一个PR?

呃…我不知道怎么把多个改动合并在一块儿提交,两个昵称改动,改动的是不同位置的文件

是我看岔了,原来是不同位置的data.json
你PR应该是在浏览器上访问github网站上去操作的吧(应该是的吧?)
那你可以在好几个 commit 之后,再编写一个 pr 请求吧?
你现在应该是每次 commit 之后都写了一个 pr 吧?
如果你提交 pr 之后,发现又有新的改动,可以再这个 pr 里直接添加新的修改内容,无需再写一个新的 pr(具体百度)

@1055169799
Copy link
Contributor Author

有个小建议,像890的添加蓝砚英文名,和889的修改集录时间这些等,
这些都是小改动,完全可以合在同一个PR里面一起提交。
还有你891 892两个都是改梦见月昵称?改动内容都一模一样,考虑关掉其中一个PR?

呃…我不知道怎么把多个改动合并在一块儿提交,两个昵称改动,改动的是不同位置的文件

是我看岔了,原来是不同位置的data.json
你PR应该是在浏览器上访问github网站上去操作的吧(应该是的吧?)
那你可以在好几个 commit 之后,再编写一个 pr 请求吧?
你现在应该是每次 commit 之后都写了一个 pr 吧?
如果你提交 pr 之后,发现又有新的改动,可以再这个 pr 里直接添加新的修改内容,无需再写一个新的 pr(具体百度)

对,我在浏览器上直接点右上角的编辑,改完以后点绿色的提交按钮,然后填写修改的说明,再点击绿色的提交就完成了

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.

2 participants