Skip to content
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

Fully migrate eth-json-rpc-provider into core monorepo #1551

Closed
xinnanyemm opened this issue Jul 25, 2023 · 1 comment
Closed

Fully migrate eth-json-rpc-provider into core monorepo #1551

xinnanyemm opened this issue Jul 25, 2023 · 1 comment
Assignees
Labels

Comments

@xinnanyemm
Copy link

xinnanyemm commented Jul 25, 2023

We would like to migrate the @metamask/eth-json-rpc-provider library into this repo to make it easier to maintain.

See the following checklist for instructions on accomplishing this: #1551 (comment)

@desi desi added the Epic label Sep 18, 2023
@MajorLift MajorLift self-assigned this Sep 21, 2023
@MajorLift MajorLift linked a pull request Sep 27, 2023 that will close this issue
4 tasks
MajorLift added a commit to MetaMask/eth-json-rpc-provider that referenced this issue Sep 28, 2023
…of migration (#28)

## Description
- This PR aligns the ESLint and Prettier packages and configs in this repo with the core monorepo.
- This is a preparation step for migrating the `eth-json-rpc-provider` package into the core monorepo.
- Originally, the plan was to decrement the TypeScript version to align with the core monorepo, but following [this discussion](#27), we've opted to use TypeScript version v4.8.4, and preserve the compiler options in this package that aren't enabled in the core monorepo.

## References
- See MetaMask/core#1551
- Closes MetaMask/core#1683
- Closes MetaMask/core#1682 (is superseded by this PR)
MajorLift added a commit that referenced this issue Sep 29, 2023
…of migration (#28)

## Description
- This PR aligns the ESLint and Prettier packages and configs in this repo with the core monorepo.
- This is a preparation step for migrating the `eth-json-rpc-provider` package into the core monorepo.
- Originally, the plan was to decrement the TypeScript version to align with the core monorepo, but following [this discussion](MetaMask/eth-json-rpc-provider#27), we've opted to use TypeScript version v4.8.4, and preserve the compiler options in this package that aren't enabled in the core monorepo.

## References
- See #1551
- Closes #1683
- Closes #1682 (is superseded by this PR)
MajorLift added a commit that referenced this issue Oct 2, 2023
## Explanation

This is the first of two PRs that will migrate
[`eth-json-rpc-provider`](https://github.com/MetaMask/eth-json-rpc-provider)
into the core monorepo by following these steps:
#1079 (comment).

- This PR focuses on migrating the git history of the original package. 
- `eth-json-rpc-provider` is moved into a temporary directory in core
(`merged-packages/`) with git history fully intact.
- The second PR will remove the `merged-packages/` directory and
integrate `eth-json-rpc-provider` into the `packages/` directory,
resolving any issues arising from applying core monorepo
linter/compiler/build settings to the newly added package.
  
## Blocked by
- [x] MetaMask/eth-json-rpc-provider#29

## References

- Partially implements #1551
- See MetaMask/eth-json-rpc-provider#28 for
preparatory steps taken in original repo.

## Changelog

N/A since `@metamask/eth-json-rpc-provider` isn't usable from the
monorepo as of this PR.

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate

---------

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: Mark Stacey <[email protected]>
Co-authored-by: Alex Donesky <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions <[email protected]>
Co-authored-by: legobeat <[email protected]>
Co-authored-by: legobt <[email protected]>
Co-authored-by: Maarten Zuidhoorn <[email protected]>
@MajorLift
Copy link
Contributor

MajorLift commented Oct 3, 2023

Migration Checklist

Phase A: Preparation in source repo

1. Add migration notice to README

<table><tr><td><p align="center"><b>⚠️ PLEASE READ ⚠️</b></p><p align="center">This package is currently being migrated to our <a href="https://github.com/MetaMask/core"><code>core</code></a> monorepo. Please do not make any commits to this repository while this migration is taking place, as they will not be transferred over. Also, please re-open PRs that are under active development in the core repo.</p></td></tr></table>

2. Disable dependabot dependency upgrades.

3. Add source repo to ZenHub workspace repo filter so that issues/PRs show up on the board.

4. Align TypeScript, ESLint, Prettier dependency versions and configurations with the core monorepo.

5. Add any missing files required in subpackages of the core monorepo (e.g. LICENSE).

6. If the package name is not prefixed by the @metamask/ namespace, rename the package.

  • Modify the "name" field in package.json.
  • Update the title of the README.md.
  • Add a CHANGELOG entry for the rename.

7. Create a new release of the migration target from the source repo


Phase B: Staging from merged-packages/

0. Move the migration target into a temporary directory in core (merged-packages/).

1. Port tags

  • See: Add script for tag porting when migrating libraries into the core monorepo #1800
    • Port the tags locally with the correct package name prefixes.
    • Push the ported tags to a fork of the core monorepo for testing.
    • Verify that the tag diff links in CHANGELOG are working.
    • The diff between last tag before migration and first one after will always include the entire history of the monorepo.
      • TODO: The correct diff can be derived using git log --ancestry-path. If GitHub starts supporting compare links that use --ancestry-path, the broken diff link in the changelog should be fixed.
    • Push the ported tags to the core repo.
    • Verify that the tag diff links in CHANGELOG are working.
    • Identify and create tags for release commits that were untagged in the original repo.

2. Remove files and directories that will be replaced by files in the monorepo root.

3. Replace config files.

  • eth-json-rpc-provider migration - B2: Replace typescript, jest config files. #1765
    • Update tsconfig*.json, typedoc.json, jest.config.js to extend from corresponding files in root. Copy contents of corresponding files in other non-root packages.
    • Keep TypeScript compiler flags and compilation target.
    • Add tsconfig reference paths for non-root packages that are upstream dependencies of the migration target.
    • Keep Jest coverage threshold values.
    • Add deepmerge as a devDependency.

4. Align dependencies and build scripts with monorepo.

  • eth-json-rpc-provider migration - B3: Align dependencies with monorepo. #1766
    • Remove redundant build scripts that are already listed in the root package.json (including prepack)
    • Identify validator fixes for CHANGELOG by navigating to merged-packages/<package-name>, running ../../scripts/validate-changelog.sh @metamask/<package-name>, and applying the diffs.
    • Remove redundant dependencies that are already listed in the root package.json.
      • Exception: do not remove TypeScript.
    • Align dependency versions with other non-root packages.
      • If migration target dependency version is ahead: Decrement to match monorepo packages.
      • If it's behind:
        • Bump if it's an internal dependency (i.e. the dependency is another sub-package in the monorepo).
        • If it's external, bump only if there are no resulting breaking changes that need to be resolved.

5. Add exception for non-MIT license.

  • add exception in the constraints.pro for non MIT license #1888
    • If migration target uses a non-MIT license, add exception entries in the root constraints.pro file.
      • Add 2 rules to license section of constraints.pro: Exclude (\=) from MIT rule and include in ISC rule (==)
    • Make sure the new rule doesn't break any of the existing package.json files by running yarn constraints.

6. Update README

  • Update contents of migration target's README to reflect new status as a non-root package in the monorepo.
    • Preserve package intro sentence/paragraph.
    • Add/modify "Installation" section.
    • Preserve "Usage" section.
    • Remove "Test", "Build" and other instructions on common development tasks.
    • Add "Contributing" section.

Phase C: Integration into packages/

1. The big leap

  • Move migration target from merged-packages/ to packages/.
  • Run yarn install in the root directory.
  • Check that all tests are passing in migration target by running yarn workspace @metamask/<package-name> test.

2. Update downstream repos

  • IMPORTANT Add reference paths for migration target in the root tsconfig.json and tsconfig.build.json files.
    • This step is essential to avoid build failure for the migration target during release workflow.
  • Add tsconfig reference paths for migration target in downstream packages.
  • Bump migration target version in downstream packages and root.

3. Linter fixes

  • Apply yarn constraints fixes to migration target package.json file: yarn constraints --fix (run twice).
  • Add the changelog:validate build script.

4. Resolve or TODO downstream errors

5. Finalize merge

  • Check that all tests are passing in all subpackages of core and CI.
  • Double-check that dependency version bumps or other changes made to main while the PR was open are correctly merged and reflected.
  • Merge packages/<package-name> directory into core main branch.

Phase D: Release and Clean-up

Source repo

  • Transfer open issues from the source repo into the core monorepo using the Transfer issue feature.
  • For open PRs in the source repo, lock conversation (with no reason provided), and leave a comment requesting that authors reopen the PR in core with a link pointing to the discussion in the original PR.
    • For important PRs, manually migrate into core or create tickets for follow-up.
This library has now been migrated into the [core monorepo](https://github.com/metamask/core/). This PR has been locked and this repo will be archived shortly. Going forward, releases of this library will only include changes made in the core repo.
- Please push this branch to core and open a new PR there.
- Optionally, add a link pointing to the discussion in this PR to provide context.
  • Leave a note in the source repo's README announcing the migration and pointing to core.
<table><tr><td><p align="center"><b>⚠️ PLEASE READ ⚠️</b></p><p align="center">This package has been migrated to our <a href="https://github.com/MetaMask/core"><code>core</code></a> monorepo, and this repository has been archived. Please note that all future development and feature releases will take place in the <a href="https://github.com/MetaMask/core"><code>core</code></a> repository.</p></td></tr></table>
  • Sunset GitHub Pages docs and leave link pointing to migrated docs in core.
  • Archive the source repo to prevent any changes from being pushed to it going forward.

Core

  • Add migration target to README dependency graph using the generate-dependency-graph build script.
  • Fix downstream errors that were marked with @ts-expect-error TODO: during the migration process.
  • Use the yarn create-release-branch tool to publish a release of core with a new version for the migrated package and any updated downstream packages.
    • Record any changes made to packages during the migration process in their respective CHANGELOGs.
  • Publish typedocs with GitHub Pages.

MajorLift added a commit that referenced this issue Oct 5, 2023
…epo. (#1766)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 3. Align dependencies with monorepo.
- [x] Remove redundant dependencies that are already listed in the root
package.json.
- [x] Bump remaining dependency versions so that they align with other
non-root packages.

***

See #1551 (comment)
for an outline of the entire process.

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 5, 2023
…s.pro` for ISC license. (#1767)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 4. Add exception for non-MIT license.
- [x] If migration target uses a non-MIT license, add exception entries
in the root `constraints.pro` file.
- [x] Make sure the new rule doesn't break any of the existing
package.json files by running `yarn constraints`.

***

See #1551 (comment)
for an outline of the entire process.

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 5, 2023
…rectories. (#1764)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 1. Remove files and directories that will be replaced by
corresponding files in the monorepo root.
- [x] **Remove**: `.github/`, `.git*`, `scripts/`, `.depcheckrc.json`,
`.yarn/`, `.yarnrc.yml`, `yarn.lock`, `.editorconfig`, `.eslint*`,
`.prettier*`, `.nvm*`.
- [x] **Keep**: `src/`, `tests/`, `CHANGELOG.md`, `LICENSE`,
`package.json`, `README.md`, `jest.config.js`, `tsconfig*.json`,
`typedoc.json`.

***

See #1551 (comment)
for an outline of the entire process.

## Blocked by
- [x] #1790

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 5, 2023
…ig files. (#1765)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 2. Replace config files.
- [x] Update `tsconfig*.json`, `typedoc.json`, `jest.config.js` to
extend from corresponding files in root. Copy contents of corresponding
files in other non-root packages.
  - [x] Keep TypeScript compiler flags. 
- [x] Add tsconfig reference paths for non-root packages that are
upstream dependencies of the migration target.
  - [x] Keep Jest coverage threshold values. 
  - [x] Add `deepmerge` as a devDependency.

***

See #1551 (comment)
for an outline of the entire process.

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 11, 2023
…rectories. (#1764)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 1. Remove files and directories that will be replaced by
corresponding files in the monorepo root.
- [x] **Remove**: `.github/`, `.git*`, `scripts/`, `.depcheckrc.json`,
`.yarn/`, `.yarnrc.yml`, `yarn.lock`, `.editorconfig`, `.eslint*`,
`.prettier*`, `.nvm*`.
- [x] **Keep**: `src/`, `tests/`, `CHANGELOG.md`, `LICENSE`,
`package.json`, `README.md`, `jest.config.js`, `tsconfig*.json`,
`typedoc.json`.

***

See #1551 (comment)
for an outline of the entire process.

## Blocked by
- [x] #1790

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 12, 2023
…s.pro` for ISC license. (#1767)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 4. Add exception for non-MIT license.
- [x] If migration target uses a non-MIT license, add exception entries
in the root `constraints.pro` file.
- [x] Make sure the new rule doesn't break any of the existing
package.json files by running `yarn constraints`.

***

See #1551 (comment)
for an outline of the entire process.

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 12, 2023
…rectories. (#1764)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 1. Remove files and directories that will be replaced by
corresponding files in the monorepo root.
- [x] **Remove**: `.github/`, `.git*`, `scripts/`, `.depcheckrc.json`,
`.yarn/`, `.yarnrc.yml`, `yarn.lock`, `.editorconfig`, `.eslint*`,
`.prettier*`, `.nvm*`.
- [x] **Keep**: `src/`, `tests/`, `CHANGELOG.md`, `LICENSE`,
`package.json`, `README.md`, `jest.config.js`, `tsconfig*.json`,
`typedoc.json`.

***

See #1551 (comment)
for an outline of the entire process.

## Blocked by
- [x] #1790

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 12, 2023
…ig files. (#1765)

## Explanation

This PR implements the following incremental step in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging in `migrated-packages/`
#### 2. Replace config files.
- [x] Update `tsconfig*.json`, `typedoc.json`, `jest.config.js` to
extend from corresponding files in root. Copy contents of corresponding
files in other non-root packages.
  - [x] Keep TypeScript compiler flags. 
- [x] Add tsconfig reference paths for non-root packages that are
upstream dependencies of the migration target.
  - [x] Keep Jest coverage threshold values. 
  - [x] Add `deepmerge` as a devDependency.

***

See #1551 (comment)
for an outline of the entire process.

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

N/A

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
MajorLift added a commit that referenced this issue Oct 13, 2023
## Explanation

This PR implements the following incremental steps in the process for
migrating `eth-json-rpc-provider` into the core monorepo:

***

### Phase B: Staging from `merged-packages/`

#### 5. Port tags 
  - See: #1800
  
<details>  
  <summary>Push ported tags to core repo</summary>
  
- [x]
https://github.com/MetaMask/core/releases/tag/@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/releases/tag/@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/releases/tag/@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/releases/tag/@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/releases/tag/@metamask/[email protected]
</details>

<details>
<summary>Verify that the tag diff links in CHANGELOG are
working</summary>
  
- [x] **WONTFIX**:
https://github.com/MetaMask/core/compare/@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/compare/@metamask/[email protected]...@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/compare/@metamask/[email protected]...@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/compare/@metamask/[email protected]...@metamask/[email protected]
- [x]
https://github.com/MetaMask/core/compare/@metamask/[email protected]...@metamask/[email protected]
</details>

### Phase C: Integration into `packages/`

#### 1. The big leap
- [x] **Move migration target from `migrated-packages/` to
`packages/`.**
- [x] Run `yarn install` in the root directory.
- [x] Check that all tests are passing in migration target by running
`yarn workspace @metamask/<package-name> test`.

#### 2. Update downstream repos
- [x] Add tsconfig reference paths for migration target in downstream
packages and root.
- [x] Bump migration target version in downstream packages and root.

#### 3. Linter fixes
- [x] Apply yarn constraints fixes to migration target package.json
file: `yarn constraints --fix` (run twice).
- [x] Identify validator fixes for CHANGELOG using `yarn workspace
@metamask/<package-name> changelog:validate` and apply the diffs.

#### 4. Resolve downstream errors
- [x] #1653
  - If introducing the migration target breaks any downstream repos:
    - [x] Resolve simple errors
- [x] Mark and ignore complex errors using `@ts-expect-error TODO:`
annotations.
- [x] Create a separate issue for resolving the marked errors as soon as
the migration is completed.

#### 5. Finalize merge
- [x] Check that all tests are passing in all subpackages of core and
CI.
- [x] Merge `packages/<package-name>` directory into core main branch.

***

See #1551 (comment)
for an outline of the entire process.

## Next Steps

- The next PR(s) will implement the final steps of the migration process
(D-1 in the migration checklist).

## Blocked by
- Dependencies:
  - [x] typescript bump: #1718
- [x] `@metamask/utils` bump: #1639
- Downstream type errors:
  - [x] #1653
- [ ] MetaMask/eth-json-rpc-provider#14
(ignored)
  - [ ] MetaMask/utils#140 (ignored)
- Tag porting:
  - [x] #1802
- [x] "Unreleased" tag diff link shows entire history of core:
https://github.com/MetaMask/core/compare/@metamask/[email protected]

## References

- Contributes to #1685
- Contributes to #1551

## Changelog

<!--
If you're making any consumer-facing changes, list those changes here as
if you were updating a changelog, using the template below as a guide.

(CATEGORY is one of BREAKING, ADDED, CHANGED, DEPRECATED, REMOVED, or
FIXED. For security-related issues, follow the Security Advisory
process.)

Please take care to name the exact pieces of the API you've added or
changed (e.g. types, interfaces, functions, or methods).

If there are any breaking changes, make sure to offer a solution for
consumers to follow once they upgrade to the changes.

Finally, if you're only making changes to development scripts or tests,
you may replace the template below with "None".
-->

### `@metamask/eth-json-rpc-provider`

- **ADDED**: Migrated into the core monorepo.

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
kanthesha added a commit that referenced this issue Oct 20, 2023
#1875)

## Explanation

<!--
Thanks for your contribution! Take a moment to answer these questions so
that reviewers have the information they need to properly understand
your changes:

* What is the current state of things and why does it need to change?
* What is the solution your changes offer and how does it work?
* Are there any changes whose purpose might not obvious to those
unfamiliar with the domain?
* If your primary goal was to update one package but you found you had
to update another one along the way, why did you do so?
* If you had to upgrade a dependency, why did you do so?
-->
In the process of migrating json-rpc-engine package into core monorepo,
in this PR, we are remove files and directories that will be replaced by
files in the monorepo root.
Remove: .github/, .git*, scripts/, .depcheckrc.json, .yarn/,
.yarnrc.yml, yarn.lock, .editorconfig, .eslint*, .prettier*, .nvm*.

## References

<!--
Are there any issues that this pull request is tied to? Are there other
links that reviewers should consult to understand these changes better?

For example:

* Fixes #12345
* Related to #67890
-->
- See #1551 

## Checklist

- [ ] I've updated the test suite for new or updated code as appropriate
- [ ] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [ ] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
@mcmire mcmire changed the title Migrate eth-json-rpc-provider libraries into core monorepo Migrate eth-json-rpc-provider library into core monorepo Oct 20, 2023
@mcmire mcmire changed the title Migrate eth-json-rpc-provider library into core monorepo Migrate eth-json-rpc-provider library into core monorepo Oct 20, 2023
@mcmire mcmire changed the title Migrate eth-json-rpc-provider library into core monorepo Fully migrate eth-json-rpc-provider into core monorepo Oct 20, 2023
MajorLift added a commit that referenced this issue Nov 7, 2023
…onorepo subpackage (#1990)

## Explanation

Adds an "Installation" and "Contributing" section to the `@metamask/json-rpc-engine` README, as it has been migrated into the monorepo.

In future migrations, this step will be performed in "Phase B" (see #1551 (comment)), while the package is being staged in the `merged-packages/` temporary directory.

## References

- Partially implements #1981.

## Changelog

N/A
MajorLift added a commit that referenced this issue Nov 8, 2023
## Explanation

- Fixes missing `json-rpc-middleware-stream` entries in root tsconfig
files that causes build failure during release workflow.

- Will add note in migration process guide
(#1551 (comment))
about the importance of step C-2.

## References

- See #1552

## Checklist

- [x] I've updated the test suite for new or updated code as appropriate
- [x] I've updated documentation (JSDoc, Markdown, etc.) for new or
updated code as appropriate
- [x] I've highlighted breaking changes using the "BREAKING" category
above as appropriate
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants