-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
chore: add Solana shared utilities and constants #28269
Conversation
CLA Signature Action: All authors have signed the CLA. You may need to manually re-run the blocking PR check if it doesn't pass in a few minutes. |
I have read the CLA Document and I hereby sign the CLA |
b24978d
to
e2b28f1
Compare
@metamaskbot update-policies |
Policy update failed. You can review the logs or retry the policy update here |
f465505
to
7f23be9
Compare
dcfa882
to
f02f41f
Compare
f02f41f
to
b882c31
Compare
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@solana/web3.js@1.95.3 |
b882c31
to
6b6fc64
Compare
👍 Dependency issues cleared. Learn more about Socket for GitHub ↗︎ This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. |
f741ec5
to
36277ef
Compare
@metamaskbot update-policies |
Policy update failed. You can review the logs or retry the policy update here |
36277ef
to
df2fcd1
Compare
@metamaskbot update-policies |
Policies updated. 🧠 Learn how: https://lavamoat.github.io/guides/policy-diff/#what-to-look-for-when-reviewing-a-policy-diff |
Builds ready [aa3a372]
Page Load Metrics (2161 ± 125 ms)
Bundle size diffs [🚨 Warning! Bundle size has increased!]
|
Description
As we look to simplify snaps ability to support multiple chains we find that some code needs to be included in the main extension to make sure it can be easily imported throughout. In this PR we add utility functions similar to the Bitcoin ones but for Solana.
Manual testing steps
Pre-merge author checklist
Pre-merge reviewer checklist