search

libi / dcron

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

add cron to dcron and splite e2e test cases from other test cases. #81

Closed dxyinme closed 6 months ago

dxyinme commented 6 months ago

etcd driver 之前的heartbeat写的有点问题，也修复了。

libi commented 6 months ago

👍👍 我这两天就看看

libi commented 6 months ago

这个合并以后也是直接作为broken版本 要不要搞一个v2 然后入口函数直接重构

dxyinme commented 6 months ago

这个合并以后也是直接作为broken版本 要不要搞一个v2 然后入口函数直接重构

感觉不是很需要作为broken版本，因为API啥的都没什么变化。搞个V2也可以，不过我觉得可以等到把cron的那些什么Entities(), GetJob()之类的函数移植过来之后再弄

libi commented 6 months ago

好吧 那就先合并主版本吧

dxyinme commented 6 months ago

好吧 那就先合并主版本吧

image 我们以后合并的时候都用这个squash merge叭，这样可以将commit压缩的少一点

libi commented 6 months ago

好的

libi commented 6 months ago

测试脚本可能需要改下， 我看你加上了 e2e 测试，看看生成的coverage用哪个呀 还是能两个都可以用。

dxyinme commented 6 months ago

测试脚本可能需要改下， 我看你加上了 e2e 测试，看看生成的coverage用哪个呀 还是能两个都可以用。

这个我研究一下，看看能不能把他们合并在一起

dxyinme commented 6 months ago

测试脚本可能需要改下， 我看你加上了 e2e 测试，看看生成的coverage用哪个呀 还是能两个都可以用。

我还是把他们合在一起了，单纯e2e的话生成的coverage是空的，而且也会丢失很多覆盖率。

codecov-commenter commented 6 months ago

Welcome to Codecov :tada:

Once merged to your default branch, Codecov will compare your coverage reports and display the results in this comment.

Thanks for integrating Codecov - We've got you covered :open_umbrella:

dxyinme commented 6 months ago

OK搞定了