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

Binance websocket #143

Merged
merged 9 commits into from
Jul 16, 2018
Merged

Binance websocket #143

merged 9 commits into from
Jul 16, 2018

Conversation

ermalguni
Copy link
Contributor

No description provided.

@thrasher-
Copy link
Collaborator

Awesome work @ermalguni!

Regarding commit: thrasher-@4bb2e5d

This breaks Travis and people who haven't setup a custom gocryptotrader directory. It would be best to revert this.

@ermalguni
Copy link
Contributor Author

@thrasher- thanks. Will revert the commit then and update the PR.

@ermalguni
Copy link
Contributor Author

@thrasher- reverted the imports. It seems that there is still sth weird going on though.

@codecov-io
Copy link

Codecov Report

Merging #143 into master will decrease coverage by 0.12%.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #143      +/-   ##
==========================================
- Coverage   37.12%   36.99%   -0.13%     
==========================================
  Files         101      102       +1     
  Lines       15878    15931      +53     
==========================================
  Hits         5894     5894              
- Misses       9465     9519      +54     
+ Partials      519      518       -1
Impacted Files Coverage Δ
exchanges/binance/binance.go 74.55% <ø> (ø) ⬆️
exchanges/binance/binance_wrapper.go 0% <0%> (ø) ⬆️
exchanges/binance/binance_websocket.go 0% <0%> (ø)

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 4fadc6f...6c79db4. Read the comment docs.

@thrasher-
Copy link
Collaborator

Thanks @ermalguni!

@thrasher- thrasher- merged commit 42ea6ba into thrasher-corp:master Jul 16, 2018
vyloy added a commit to vyloy/gocryptotrader that referenced this pull request Jul 18, 2018
* 'master' of https://github.com/thrasher-/gocryptotrader:
  Fixes theme picker reference error after changes in https://github.com/thrasher-/gocryptotrader/pull/154
  Improved code quality (thrasher-corp#154)
  Improve request package and adjust bitfinex rate limiter
  Bump Travis golang version
  Update exchange wrapper template file
  Binance websocket (thrasher-corp#143)
  Adds support for new Huobi authorised requests and a tool for ecdsa keys
  Use WebsocketEvent in ws client and add timestamp to Tickers
  Fix race condition in request
  Bithumb replace deprecated API to new one
  Improve Bithumb API error handling
  Add bank details support
  Update README.md
  Skip BTCC tests as REST API is not available after infrastructure update
  Rename GDAX -> CoinbasePro
  Hotfix/web dependencies (thrasher-corp#149)
  Fixed some typos in binance types and updated binance trading symbols in config example (thrasher-corp#148)
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.

3 participants