-
Notifications
You must be signed in to change notification settings - Fork 39
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
Get ABI from Sourcify #42
Conversation
🦋 Changeset detectedLatest commit: 19445bd The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
Nice work!
I think we should prefer to use official SDK and if something's missing we should help to shape it. For now I would consider this blocked until CJS support is there. CC: @kuzdogan |
Hey @kuzdogan, I talked with @krzkaczor now, and we'll use the |
@krzkaczor how would you see |
bf04bff
to
1854532
Compare
1854532
to
0173d3a
Compare
More or less yes. Simply put if ABI doesn't exist already just fail with instructions on how to provide it. |
11afd71
to
7a46e9e
Compare
54fa94b
to
c025c72
Compare
5582fee
to
46277bd
Compare
@@ -3,7 +3,5 @@ dist | |||
yarn-error.log | |||
.DS_Store | |||
*.tsbuildinfo | |||
packages/tests/eth-sdk |
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.
Moved to a separate gitignore file
All right, this is ready for review. |
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.
Missing changeset
What's changed?
I added two ways to grab metadata from Sourcify. It's not production-ready yet, but we can work with it.
What needs fixing here?
@ethereum-sourcify/contract-call-decoder
built code is ESM, so I had to make some changes to our test setup and change "type" to "module" in package.json files. This obviously cannot be published, because it would break everything. We can find a work around to use the decoder or maybe leverage.mts
extension from TypeScript 4.5Questions
Further work