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

重构了一下,改得有点多。 #4

Open
wants to merge 13 commits into
base: master
Choose a base branch
from

Conversation

cupen
Copy link

@cupen cupen commented Apr 19, 2019

  • 构建方案改用 go module (兼容 gopath + vendor),方便 fork 在本地构建。
  • 发布改用 docker 以免编译/运行环境差异过大 , 见 make release .
  • 配置文件改用 toml 语法,代码层面就不必去手工解析,可以去掉一些 strings.trim 之类的代码。
  • 引入 validator, 在 struct tag 里写好断言就行,减少硬编码错误检查。
  • 支持 bearychat
  • 支持 feishu
  • 支持一个事件触发多个通知,比如可同时发到 slack 和 mail 等等,配置多少就发多少。

@cupen
Copy link
Author

cupen commented Apr 19, 2019

忘了说,我加了 make release ,可直接发布 linux 版的二进制包,见 ./build 目录。
其他平台的支持是不需要的,比如 windows 下用不了 supervisor, mac 也没人拿来跑 7×24 小时的程序。

@ouqiang
Copy link
Owner

ouqiang commented Apr 20, 2019

感谢,我有空的时候看一下。

cupen added 5 commits May 22, 2020 21:04
* 支持多目标通知
* 配置文件改用 toml 格式,不再需要手动 trim
* 引入自动 validate,不再手动校验配置错误。
* 增加飞书支持
@cupen cupen changed the title 改用 go mod 、增加文件日志(便于调试)、bearychat 支持 重构了一下,改得有点多。 Mar 31, 2021
@cupen
Copy link
Author

cupen commented Mar 31, 2021

@ouqiang 有空看下?

@cupen
Copy link
Author

cupen commented Sep 9, 2021

@ouqiang Hi 你还维护这个项目不? 不介意的话, 我会继续在我的分支上胡改瞎改.

@cupen cupen force-pushed the master branch 3 times, most recently from a02cbd0 to fb0be1c Compare September 9, 2021 08:43
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.

None yet

2 participants