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

Resolve "Mark kraken.spot.KrakenSpotWSClientV1 as deprecated" #201

Conversation

btschwertfeger
Copy link
Owner

As the related issue notes - the websocket API v1 will most probably be replaced by API v2, so marking it as deprecated is required.

@btschwertfeger btschwertfeger added the Spot Topic related to Spot trading label Mar 10, 2024
@btschwertfeger btschwertfeger added this to the Future Release milestone Mar 10, 2024
@btschwertfeger btschwertfeger self-assigned this Mar 10, 2024
@btschwertfeger btschwertfeger linked an issue Mar 10, 2024 that may be closed by this pull request
Copy link

codecov bot commented Mar 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.45%. Comparing base (8387327) to head (2b7d3aa).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #201      +/-   ##
==========================================
+ Coverage   88.44%   88.45%   +0.01%     
==========================================
  Files          24       24              
  Lines        1973     1975       +2     
==========================================
+ Hits         1745     1747       +2     
  Misses        228      228              
Flag Coverage Δ
unittests 88.45% <100.00%> (+0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
kraken/spot/websocket_v1.py 97.20% <100.00%> (+0.03%) ⬆️

@btschwertfeger btschwertfeger merged commit f08992f into master Mar 10, 2024
32 checks passed
@btschwertfeger btschwertfeger deleted the 138-introduce-the-deprecated-decorator-to-mark-spot-websocket-api-v1-as-deprecated branch March 10, 2024 11:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Spot Topic related to Spot trading
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mark kraken.spot.KrakenSpotWSClientV1 as deprecated
1 participant