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

feat: 获取剩余@全员次数 & fix #126

Merged
merged 4 commits into from
Nov 16, 2024
Merged

Conversation

icarus-ai
Copy link
Contributor

No description provided.

@icarus-ai
Copy link
Contributor Author

😄

@icarus-ai icarus-ai changed the title fmt: import feat: 获取剩余@全员次数 & fix & fmt Nov 16, 2024
@icarus-ai
Copy link
Contributor Author

嘤嘤嘤

@Redmomn
Copy link
Collaborator

Redmomn commented Nov 16, 2024

不接受无意义的format

@icarus-ai
Copy link
Contributor Author

不接受无意义的format

回滚了 fmt

@icarus-ai icarus-ai changed the title feat: 获取剩余@全员次数 & fix & fmt feat: 获取剩余@全员次数 & fix Nov 16, 2024
@icarus-ai
Copy link
Contributor Author

还差点 吃饱了再搞 errors.Wrap

bool isArk = 12;
string arkName = 13;
bool isFinish = 14;
optional string refer = 2;
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

解释一下为什么这么改

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

解释一下为什么这么改

miraigo就是这么写的 Mrs4s: MiraiGo/client/pb/oidb/oidb0xbcb.proto

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

你应该问为什么之前不是这么写的😊

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有区别吗?

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

再有这种无意义的提交,我不介意把你block掉

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有时间格式化,不如去加个ci配置,让他自动化

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有时间格式化,不如去加个ci配置,让他自动化

ide 自动 关我什么事

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

有区别吗?

改错代码 (误删optional) 回滚一下 可 ?

@icarus-ai
Copy link
Contributor Author

这条提交到此为止

@Redmomn Redmomn merged commit 0f7f970 into LagrangeDev:master Nov 16, 2024
2 checks passed
@Redmomn
Copy link
Collaborator

Redmomn commented Nov 16, 2024

optional我就当你是误删,但是因为一个自动格式化就开过来一条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