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

make dcron running locally, update cron test, add GetJobs / GetJob function for dcron. #89

Merged
merged 72 commits into from
Feb 6, 2024

Conversation

dxyinme
Copy link
Collaborator

@dxyinme dxyinme commented Feb 3, 2024

No description provided.

dxyinme and others added 24 commits December 31, 2023 00:30
This reverts commit 99b2d82.
…i#80)

* add example app to readme

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* update

* update

* update

* update

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (libi#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments
@codecov-commenter
Copy link

Codecov Report

Attention: 17 lines in your changes are missing coverage. Please review.

Comparison is base (0ff369b) 84.97% compared to head (106db08) 85.75%.

Files Patch % Lines
dcron.go 66.00% 15 Missing and 2 partials ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop      #89      +/-   ##
===========================================
+ Coverage    84.97%   85.75%   +0.77%     
===========================================
  Files           17       17              
  Lines         1278     1102     -176     
===========================================
- Hits          1086      945     -141     
+ Misses         160      125      -35     
  Partials        32       32              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@dxyinme dxyinme requested a review from libi February 6, 2024 02:46
@libi libi merged commit c9eca56 into libi:develop Feb 6, 2024
3 checks passed
libi added a commit that referenced this pull request Mar 13, 2024
* chore: use `redis.UniversalClient` instead of `*redis.Client` (#88)

* make dcron running locally, update cron test, add GetJobs / GetJob function for dcron. (#89)

* 增加code coverage, 修复GetJob bug,增加devcontainer方便开发者 (#91)

* add example app to readme

* add NodeID function into dcron

* Deps: updated github.com/go-redis/redis/v8 to github.com/redis/go-redis/v9 (#73)

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* add robfig/cron to dcron (#75)

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* update

* update

* update

* update

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* Add comments, add split the E2E test cases and other test cases. (#80)

* add example app to readme

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* update

* update

* update

* update

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* merge e2e test and normal test

* move e2etest to origin path

* add timeout to avoid pipeline timeout

* add getjobs related function

* update

* remove redis config in test workflow

* update

* update chain test in windows

* update action version for node 16 -> node 20

* add test for dcron locally

* update readme

* update

* fix code bug

* Update devcontainer and etcd driver test

---------

Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>

---------

Co-authored-by: AHdark <ahdark@outlook.com>
Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>
dxyinme added a commit that referenced this pull request Apr 24, 2024
* chore: use `redis.UniversalClient` instead of `*redis.Client` (#88)

* make dcron running locally, update cron test, add GetJobs / GetJob function for dcron. (#89)

* 增加code coverage, 修复GetJob bug,增加devcontainer方便开发者 (#91)

* add example app to readme

* add NodeID function into dcron

* Deps: updated github.com/go-redis/redis/v8 to github.com/redis/go-redis/v9 (#73)

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* add robfig/cron to dcron (#75)

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* update

* update

* update

* update

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* Add comments, add split the E2E test cases and other test cases. (#80)

* add example app to readme

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update

* update

* update

* update

* update

* update

* update

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* merge e2e test and normal test

* move e2etest to origin path

* add timeout to avoid pipeline timeout

* add getjobs related function

* update

* remove redis config in test workflow

* update

* update chain test in windows

* update action version for node 16 -> node 20

* add test for dcron locally

* update readme

* update

* fix code bug

* Update devcontainer and etcd driver test

---------

Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>

* update test coverage

* make driver withOption -> WithOption

* update go.mod

* update go.mod

* update protobuf version

* etcd to 3.5.12

* fix etcd

* fix etcd

* Revert "fix etcd"

This reverts commit a21bf33.

* update miniredis

* update

* update go.mod

* update go mod

* update go mod

---------

Co-authored-by: AHdark <ahdark@outlook.com>
Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>
libi added a commit that referenced this pull request May 20, 2024
* opt: use runtime/debug.Stack() to print stack trace of panic (#93)

* 提升测试覆盖率,增加go1.22的test action. (#94)

* chore: use `redis.UniversalClient` instead of `*redis.Client` (#88)

* make dcron running locally, update cron test, add GetJobs / GetJob function for dcron. (#89)

* 增加code coverage, 修复GetJob bug,增加devcontainer方便开发者 (#91)

* add example app to readme

* add NodeID function into dcron

* Deps: updated github.com/go-redis/redis/v8 to github.com/redis/go-redis/v9 (#73)

* add cron lib

* fix warning

* logger removal : phares1

* add robfig/cron to dcron (#75)

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* Add comments, add split the E2E test cases and other test cases. (#80)

* add example app to readme

* add NodeID function into dcron

* add cron lib

* fix warning

* logger removal : phares1

* update test workflow

* revert 1.22

* update etcd driver

* update

* fix get nodes

* update

* update for fix TAT

* Revert "update etcd driver"

This reverts commit a21ebf7.

* Revert "deps: updated go-redis to v9 (#79)"

This reverts commit 0b85b24.

* update

* update

* refact etcd

* Revert "refact etcd"

This reverts commit 049bed1.

* Revert "update"

This reverts commit 9c71fd6.

* update

* refactor etcddriver

* fix error

* update

* Revert "update"

This reverts commit 6cfcfe6.

* Revert "fix error"

This reverts commit 99b2d82.

* Revert "refactor etcddriver"

This reverts commit a576ac3.

* update

* update

* remove comments, and fix

* add comments

* merge e2e test and normal test

* move e2etest to origin path

* add timeout to avoid pipeline timeout

* add getjobs related function

* update

* remove redis config in test workflow

* update

* update chain test in windows

* update action version for node 16 -> node 20

* add test for dcron locally

* update readme

* update

* fix code bug

* Update devcontainer and etcd driver test

---------

Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>

* update test coverage

* make driver withOption -> WithOption

* update go.mod

* update go.mod

* update protobuf version

* etcd to 3.5.12

* fix etcd

* fix etcd

* Revert "fix etcd"

This reverts commit a21bf33.

* update miniredis

* update

* update go.mod

* update go mod

* update go mod

---------

Co-authored-by: AHdark <ahdark@outlook.com>
Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>

* remove 1.22 test action to increase the test coverage. (#97)

---------

Co-authored-by: coti <54785329+coticom@users.noreply.github.com>
Co-authored-by: AHdark <ahdark@outlook.com>
Co-authored-by: Ava Ackerman <withrjp@gmail.com>
Co-authored-by: libi <7769922+libi@users.noreply.github.com>
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.

4 participants