-
Notifications
You must be signed in to change notification settings - Fork 13
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
feat(simulation): update simulation models (breaking change) #632
Conversation
|
New and removed dependencies detected. Learn more about Socket for GitHub ↗︎
🚮 Removed packages: npm/@openzeppelin/defender-sdk@1.15.2, npm/axios@1.7.2, npm/rollup@4.19.0 |
👍 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. |
@shahnami can you sort the breaking tests, please? |
@tirumerla looks like this package was introduced in another PR (the one adding EAS example I think), any idea why it's complaining 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.
LGTM, if includes breaking changes I would do a quick post in the forum just in case
Summary
Testing Process
Checklist