chore(stellar-wallet-snap): migration phase C - Integration Stellar into packages + Linter and constraints fixes + update changelog - #161
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
|
All alerts resolved. Learn more about Socket for GitHub. This PR previously contained dependency changes with security issues that have been resolved, removed, or ignored. Ignoring alerts on:
|
| 'jest/no-alias-methods': 'error', | ||
| 'jest/no-commented-out-tests': 'error', | ||
| 'jest/no-disabled-tests': 'error', | ||
| 'jest/no-mocks-import': 'off', |
There was a problem hiding this comment.
stellar is import some test utils from mocks in Test file, skip this rule would not cause addtional eslint issue
|
@SocketSecurity ignore @stellar/stellar-sdk |
|
@metamaskbot publish-preview |
|
@SocketSecurity ignore-all |
| "ripemd160@npm:^2.0.1": "2.0.2", | ||
| "ws@7.4.6": "^7.5.10" | ||
| "ws@7.4.6": "^7.5.10", | ||
| "@stellar/stellar-sdk/axios@npm:1.15.0": "1.18.1", |
There was a problem hiding this comment.
1.15.0 has security concern, resolve to 1.18.1
|
@metamaskbot publish-preview |
|
Preview builds have been published. Learn how to use preview builds in other projects. Expand for full list of packages and versions. |
|
@stanleyyconsensys GG. Please try to run |




Explanation
This PR is cover the migration all steps on phase C
Move the package from merged-packages/ to packages/ (included in PR))
Linter and constraints fixes (include in the PR)
Resolve or TODO downstream errors (nth to do)
Record changes in CHANGELOG [Unreleased] (included in the PR)
Finalize merge (done)
Migration process
References
Checklist