V2 update #75

Merged

nikooo777 merged 4 commits from replace-v2 into master

2019-10-25 16:49:59 +02:00

nikooo777 commented

2019-10-10 16:55:37 +02:00

(Migrated from github.com)

lbry.go depends on itself, if imports to itself aren't upgraded to v2 it will use a weird mix of dependencies.
This PR ports every usage to V2.

Additionally, several modules were updated, I updated them all (skipped btcd because it's still broken)

I tested this with ytsync and it still works fine. jsonrpc tests also pass (though the wallet related ones added by Andrey didn't pass, probably due to differences in chains?).

lbry.go depends on itself, if imports to itself aren't upgraded to v2 it will use a weird mix of dependencies. This PR ports every usage to V2. Additionally, several modules were updated, I updated them all (skipped btcd because it's still broken) I tested this with ytsync and it still works fine. jsonrpc tests also pass (though the wallet related ones added by Andrey didn't pass, probably due to differences in chains?).

anbsky (Migrated from github.com) approved these changes 2019-10-18 07:43:43 +02:00

anbsky (Migrated from github.com) left a comment

Thanks for doing the work I should've done! I fixed TestClient_WalletList which was failing for me, others seem to run fine. All in all looks good to me.

Thanks for doing the work I should've done! I fixed `TestClient_WalletList` which was failing for me, others seem to run fine. All in all looks good to me.

No reviewers

anbsky

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: LBRYCommunity/lbry.go#75

No description provided.