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

🐛 FIX: Ignore exists seq on changes worker #253

Merged
merged 1 commit into from
Jun 27, 2022
Merged

Conversation

fengmk2
Copy link
Member

@fengmk2 fengmk2 commented Jun 27, 2022

No description provided.

@fengmk2 fengmk2 added the bug Something isn't working label Jun 27, 2022
@fengmk2
Copy link
Member Author

fengmk2 commented Jun 27, 2022

#252 (comment) ci 挂的原因
cc @atian25

@codecov
Copy link

codecov bot commented Jun 27, 2022

Codecov Report

Merging #253 (0fea590) into main (4ea0ef6) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #253   +/-   ##
=======================================
  Coverage   97.16%   97.16%           
=======================================
  Files         119      119           
  Lines        3634     3634           
  Branches      321      321           
=======================================
  Hits         3531     3531           
  Misses        103      103           
Impacted Files Coverage Δ
app/core/service/ChangesStreamService.ts 84.41% <100.00%> (ø)
app/port/UserRoleManager.ts 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c63159d...0fea590. Read the comment docs.

@fengmk2 fengmk2 merged commit d578baf into main Jun 27, 2022
@fengmk2 fengmk2 deleted the ignore-exists-since-seq branch June 27, 2022 11:53
@fengmk2
Copy link
Member Author

fengmk2 commented Jun 27, 2022

1.3.2

@atian25
Copy link
Contributor

atian25 commented Jun 27, 2022

报错原因和日志能否打印啥的,那个单测没啥日志输出来判断

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants