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

wip: Support web3 >= 1.0.0-beta.38 #95

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

wip: Support web3 >= 1.0.0-beta.38 #95

wants to merge 2 commits into from

Conversation

sunify
Copy link
Member

@sunify sunify commented Apr 24, 2019

It's a more workaround rather final solution. I think we should extract web3 stuff from core and create a separate package for that. Same for ethers. Otherwise it would be kinda hard to do #82

@sunify sunify requested a review from troggy April 24, 2019 05:47
@sunify sunify removed the request for review from troggy April 24, 2019 05:47
@codecov-io
Copy link

codecov-io commented Apr 24, 2019

Codecov Report

Merging #95 into master will decrease coverage by 4.32%.
The diff coverage is 5.26%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
- Coverage   84.21%   79.88%   -4.33%     
==========================================
  Files          12       13       +1     
  Lines         912     1034     +122     
==========================================
+ Hits          768      826      +58     
- Misses        144      208      +64
Impacted Files Coverage Δ
lib/web3/AbstractMethod.js 5% <5%> (ø)
lib/helpers.js 51.85% <5.55%> (-25.08%) ⬇️
lib/transaction.js 91.78% <0%> (+1.85%) ⬆️

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 d10a19c...8920af1. Read the comment docs.

@troggy
Copy link
Member

troggy commented Apr 24, 2019

FYI #66

@sunify
Copy link
Member Author

sunify commented Apr 24, 2019

Yeah, i saw it, thanks. Just don't want to drag all this web3 stuff into our precious library :-)

package.json Outdated Show resolved Hide resolved
@troggy
Copy link
Member

troggy commented Apr 25, 2019

why package-lock?

@sunify
Copy link
Member Author

sunify commented Apr 25, 2019

Oh, we hadn't it before?

@troggy
Copy link
Member

troggy commented Apr 25, 2019

only yarn.lock

@sunify
Copy link
Member Author

sunify commented Apr 25, 2019

Ok, looks like i missed when we started use yarn in this repo :-) Will remove

@troggy
Copy link
Member

troggy commented Apr 25, 2019

@types/web3 still uses old Web3, so typedefs of leap-core are not going to work with latest web3 :(

testing it with leapdao/bridge-ui#163

@sunify
Copy link
Member Author

sunify commented Apr 25, 2019

Fuck. Totally forgot about typings :(

@troggy
Copy link
Member

troggy commented Apr 25, 2019

if we update typings here, this will break type-compatiblity with older web3, right?

@sunify
Copy link
Member Author

sunify commented Apr 25, 2019

Not sure. Will take a look at typings and what we can do.

@troggy troggy changed the title Support web3 >= 1.0.0-beta.38 wip: Support web3 >= 1.0.0-beta.38 May 10, 2019
@troggy troggy removed the in progress label Dec 5, 2019
@sunify sunify mentioned this pull request Feb 4, 2020
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