-
Notifications
You must be signed in to change notification settings - Fork 822
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
Bugfix: Bitstamp race condition and ANX format directive issue #342
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
approved 🙋♂️
exchanges/anx/anx_live_test.go
Outdated
@@ -20,7 +20,7 @@ func TestMain(m *testing.M) { | |||
cfg.LoadConfig("../../testdata/configtest.json") | |||
anxConfig, err := cfg.GetExchangeConfig("ANX") | |||
if err != nil { | |||
log.Fatalf("Test Failed - ANX Setup() init error", err) | |||
log.Fatal("Test Failed - ANX Setup() init error", err) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👀 % 👀
Codecov Report
@@ Coverage Diff @@
## master #342 +/- ##
=======================================
Coverage 43.55% 43.55%
=======================================
Files 129 129
Lines 30991 30991
=======================================
Hits 13499 13499
Misses 16374 16374
Partials 1118 1118
Continue to review full report at Codecov.
|
…dual getfee call as needed
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK, thanks!
Description
Various bug fixes
Type of change
Please delete options that are not relevant and add an
x
in[]
as item is complete.How Has This Been Tested?
Checklist: