-
Notifications
You must be signed in to change notification settings - Fork 29
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
Add ESLint rule for BN imports #197
Conversation
packages/eslint-config/index.js
Outdated
"error", | ||
{ | ||
name: "bn.js", | ||
message: "We no longer use BN.js for big number calculations. Switch to using bignumber.js instead. Run npm run convert-bn to automatically switch BN to BigNumber in all folders" |
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.
since we use pnpm maybe we can stick to it?
@@ -1 +1 @@ | |||
export * as StreamflowDistributorSolana from "./solana"; | |||
export * as StreamflowDistributorSolana from "./solana/index.js"; |
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.
you said you were planning to migrate scoped exports here, do I miss anything here?
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 gave up on that - there are issues with v7 even with that, right now I paused that work in favour of more prio work (aligned unlocks/airdrops)
npm run convert-bn
to run on all files underpackages
or you can specify the folders e.gnpm run convert-bn -- packages/stream/solana