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

refactor isolation context for persistence facade configuration #982

Merged
merged 3 commits into from
Oct 5, 2022

Conversation

c9s
Copy link
Owner

@c9s c9s commented Oct 5, 2022

No description provided.

@bbgokarma-bot
Copy link

Welcome back! @c9s, This pull request may get 297 BBG.

@codecov
Copy link

codecov bot commented Oct 5, 2022

Codecov Report

Merging #982 (e922191) into main (4d42a61) will decrease coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #982      +/-   ##
==========================================
- Coverage   19.92%   19.91%   -0.02%     
==========================================
  Files         420      420              
  Lines       31883    31901      +18     
==========================================
  Hits         6353     6353              
- Misses      24925    24943      +18     
  Partials      605      605              
Impacted Files Coverage Δ
pkg/bbgo/bootstrap.go 0.00% <0.00%> (ø)
pkg/bbgo/environment.go 1.34% <ø> (+0.03%) ⬆️
pkg/bbgo/isolation.go 40.00% <0.00%> (-13.34%) ⬇️
pkg/bbgo/persistence.go 39.43% <0.00%> (-27.24%) ⬇️

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 4d42a61...e922191. Read the comment docs.

@c9s c9s merged commit 39247bb into main Oct 5, 2022
@c9s c9s deleted the refactor/isolation branch October 5, 2022 14:31
@bbgokarma-bot
Copy link

Hi @c9s,

Well done! 307 BBG has been sent to your polygon wallet. Please check the following tx:

https://polygonscan.com/tx/0xef6cac79969fd7c65456c2f497b1a4b4fb256973974445749d6f75ffc79b1afa

Thank you for your contribution!

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