-
Notifications
You must be signed in to change notification settings - Fork 322
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
LIVE-14029 evm importer limiting to top 100 tokens by chains #7903
base: develop
Are you sure you want to change the base?
Conversation
The latest updates on your projects. Learn more about Vercel for Git βοΈ
4 Skipped Deployments
|
1da95f7
to
e40dcab
Compare
lint
remove unused dep updatereadme clear changelog file
698e9a6
to
5e7a870
Compare
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.
Just a bunch of (I think) useless files you can remove from the lib as they will not be consumed by anyone !
Also, I think you need to update evm-tools
to consumed the EIP712 files and this new library instead of the normal cryptoassets
otherwise it would still be a peer dependency of a peer dependency
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.
Probably unnecessary to have this file in the new lib
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.
Probably unnecessary to have this file in the new lib
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.
Probably unnecessary to have this file in the new lib
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.
Probably unnecessary to have this file in the new lib
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.
Can probably be removed as well, documentation for such lib seems unnecessary
"build": "tsc && tsc -m ES6 --outDir lib-es", | ||
"prewatch": "pnpm build", | ||
"watch": "tsc --watch", | ||
"doc": "documentation readme src/** --section=API --pe ts --re ts --re d.ts", |
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.
I'd remove the doc script to prevent painful forgotten commits about this for no reason if I were you
β Checklist
npx changeset
was attached.π Description
all chains, no filtering:
evm only, no filtering by top 100:
evm only, filtering by top:
β Context
π§ Checklist for the PR Reviewers