Add snaps-cli #72
Merged
Add snaps-cli #72
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Use yargs, add watch and serve commands, Quality-of-Life stuff
* refactor to use npm conventions * add SES eval command * update file/directory structure
* Move example to examples folder * Add ping pong example * Add BLS signer example * move example deps to examples * bugfixes
Get bls-signer to a working state
Fix requestedPermissions
* Bump @metamask/eslint-config from 6.0.0 to 8.0.0 Bumps [@metamask/eslint-config](https://github.com/MetaMask/eslint-config) from 6.0.0 to 8.0.0. - [Release notes](https://github.com/MetaMask/eslint-config/releases) - [Commits](MetaMask/eslint-config@v6.0.0...v8.0.0) --- updated-dependencies: - dependency-name: "@metamask/eslint-config" dependency-type: direct:development update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> * Manually fix lint issues, simplify config Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Erik Marks <[email protected]>
Refactors the package exports (i.e. the exports of the package manifest `main` file, which is `dist/cmds/index`) such that the complete command array is exported under the name `commandModules`, and every individual command handler is exported under its name. The default export is removed.
This PR reverts #131, and completely removes the `main` field from the package manifest. We don't actually want to maintain any contract with consumers regarding our command module exports! The CLI itself is the interface. Packages that want to programmatically invoke `snaps-cli` can use `execa` or the command line.
This renames the `./development` directory to `./scripts`, which is more in line with other repositories in our organization.
This PR adds a mock `request` method and all of the `EventEmitter` methods to the mock `wallet` provider object in `eval-worker.ts`. These were previously missing, and made it fail example snaps that should succeed.
This was referenced Sep 8, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Adds the
@metamaskpackagessnaps-cliandsnap-examplesto this monorepo.Ref: