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

新增统计睡眠时间和添加协程里的异常捕获 #84

Merged
merged 16 commits into from
Dec 4, 2021
Merged

新增统计睡眠时间和添加协程里的异常捕获 #84

merged 16 commits into from
Dec 4, 2021

Conversation

guohuiyuan
Copy link
Contributor

No description provided.

main.go Outdated Show resolved Hide resolved
plugin_sleep_manage/sleep_manage.go Show resolved Hide resolved
plugin_sleep_manage/sleep_manage.go Outdated Show resolved Hide resolved
plugin_sleep_manage/model/model.go Show resolved Hide resolved
plugin_sleep_manage/sleep_manage.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
control/rule.go Outdated Show resolved Hide resolved
plugin_manager/manager.go Show resolved Hide resolved
plugin_manager/manager.go Outdated Show resolved Hide resolved
plugin_manager/manager.go Outdated Show resolved Hide resolved
@guohuiyuan
Copy link
Contributor Author

guohuiyuan commented Dec 4, 2021 via email

@fumiama
Copy link
Member

fumiama commented Dec 4, 2021

主要是我想让用户更加简单的触发help,那个打印的是服务列表+用法的全部信息,有很多功能不知道,要知道用法需要两步有点难搞

但是实在太长了...

@guohuiyuan
Copy link
Contributor Author

guohuiyuan commented Dec 4, 2021 via email

@fumiama
Copy link
Member

fumiama commented Dec 4, 2021

我会改成合并消息

可以,不过建议另外用别的关键词触发,help还是保持简洁,仅提示这个触发关键词,以及 /服务列表 + /用法 的两级查询结构。

@fumiama fumiama self-requested a review December 4, 2021 12:43
Copy link
Member

@fumiama fumiama left a comment

Choose a reason for hiding this comment

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

没有其他更改我就合并了

@guohuiyuan
Copy link
Contributor Author

guohuiyuan commented Dec 4, 2021 via email

@fumiama fumiama merged commit 793cac0 into FloatTech:master Dec 4, 2021
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