libi / dcron

轻量分布式定时任务库 a lightweight distributed job scheduler library
MIT License
422 stars 73 forks source link

mv cmd job to special_jobs #53

Closed libi closed 1 year ago

dxyinme commented 1 year ago

希望能够增加cmd job相关的测试

libi commented 1 year ago

这个实现感觉还能优化一下,主要有两个:

  1. bash 命令应该存在需要接收命令执行结果的需求,需要修改实现。
  2. 单次执行多条 bash 可以改为单条简化执行逻辑。多条可以通过多个job实现,某些场景下需要单job多bash语句,可以用bash语句的 && 。
libi commented 1 year ago

@dxyinme 帮看下这个pr还有啥问题没

dxyinme commented 1 year ago

@dxyinme 帮看下这个pr还有啥问题没

欸我提了一些review意见是没显示嘛

dxyinme commented 1 year ago

@dxyinme 帮看下这个pr还有啥问题没

淦,要request changes才行

libi commented 1 year ago

哈哈 review意见我这看不到 那另一个pr我提的意见你是不是也看不见啊。 我也一直以为review评论就会显示🤣

dxyinme commented 1 year ago

哈哈 review意见我这看不到 那另一个pr我提的意见你是不是也看不见啊。 我也一直以为review评论就会显示🤣

看不着,我之前也以为review了就会显示,好难用啊这,,,

libi commented 1 year ago

看了下文档 🤕️确实就是这么设计的 https://docs.github.com/zh/pull-requests/collaborating-with-pull-requests/reviewing-changes-in-pull-requests/reviewing-proposed-changes-in-a-pull-request

提交审查之前,你的行注释为“待处理”状态并且仅对你可见。 您可以在提交审查之前随时编辑待处理的注释。 若要取消挂起的审查(包括其所有挂起的注释),请单击已更改的代码上方的“审查更改”,然后单击“取消审查”。