-
-
Notifications
You must be signed in to change notification settings - Fork 267
Convert to monorepo #831
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
Merged
Merged
Convert to monorepo #831
Changes from all commits
Commits
Show all changes
135 commits
Select commit
Hold shift + click to select a range
be212ce
POC - Start converting to monorepo
mcmire d4a7428
Migrate from Yarn v1 to Yarn v3
Gudahtt 0da8306
Replace `setup` script with new Yarn plugin
Gudahtt 0164652
Merge branch 'main' into upgrade-to-yarn-3
Gudahtt 36afcd4
Fix typo
Gudahtt d9c87b0
Merge branch 'upgrade-to-yarn-3' into monorepo-poc
mcmire 30e86e2
Move remaining packages to packages/
mcmire 6e4ea9a
Get all tests passing
mcmire 5e2613a
Merge branch 'main' into monorepo-poc
mcmire be2bbe1
Simplify build:clean command
mcmire 25916fd
Simplify globs in tsconfigs
mcmire 8b8a688
Use __dirname in Jest config
mcmire 97d5e3f
Fix linting
mcmire 826e62d
Merge branch 'upgrade-to-yarn-3' into monorepo-poc
mcmire 416f0ad
Clean some things up
mcmire ddab44d
Simplify Jest config, move configs to each package
mcmire 2b4345a
Fix lint issues
mcmire 8ed6e03
Fix TypeDoc
mcmire 6991d90
Fix CI so that --maxWorkers works
mcmire f06ac0a
Fix tests to not hang if there are outstanding timers
mcmire 7395afc
Merge branch 'main' into monorepo-poc
mcmire c0ff9e2
Initialize changelogs for all packages
mcmire cff70d7
Add 'yarn constraints' to CI
mcmire 059d330
Oops, fix yarn.lock
mcmire 00be2cd
Fix lint issues
mcmire 7eb7e17
Put build and test after lint
mcmire 4baf34f
Add Yarn version plugin
mcmire ef9fdbd
Add changesets
mcmire fc1c892
Make some quick tweaks
mcmire 38a72df
Oops, remove .changeset directory
mcmire c24f6db
Turns out we need this after all
mcmire b61ac10
Merge main
mcmire c242b76
Merge branch 'main' into monorepo-poc
mcmire 3cd59d2
Revert/restore build-lint-test.yml
mcmire 07bab4b
Fix changelog validation in CI
mcmire 0c34e53
Merge branch 'main' into monorepo-poc
mcmire afecade
Merge branch 'main' into monorepo-poc
mcmire b93feaa
Fix TypeScript and linting issues
mcmire 91de999
Merge branch 'main' into monorepo-poc
mcmire 0fa45f8
Remove symlinks for scripts in favor of using their direct paths
mcmire fbaffa9
Update README
mcmire 0963bdd
build, lint, test -> lint, build, test
mcmire 5f6205c
Fix constraints
mcmire b7f8c6d
Update package READMEs to provide description and remove Usage (for now)
mcmire a824252
Don't use 4 spaces for indentation in Markdown files
mcmire 7d1532a
Fix lint issues
mcmire 90de519
Split apart more controllers
mcmire e0dfdf1
Add --forceExit again to Jest command on CI
mcmire 7a568e8
Fix Yarn constraints to correct compare version strings
mcmire 4f98075
Extract more things from controller-utils that should be in other pac…
mcmire d294040
Regenerate yarn.lock
mcmire 045b4e3
Clean up constraints.pro
mcmire 26db827
Don't export this file
mcmire 9537b08
Fix READMEs for composable-controller and message-manager
mcmire 16f425e
Flip around some lines to make diffs easier to read
mcmire 99ed774
Update descriptions in package.json to match README
mcmire 48fbdab
Remove the need for an extra tsconfig for tests
mcmire fc07abc
Merge branch 'main' into monorepo-poc
mcmire 72d47b1
Fix exports of all packages to match current exports
mcmire 7447865
Add more constraints
mcmire ccd2e83
Bump the version to the current version
mcmire 190489d
Start all packages' versions from scratch
mcmire d9e2b7a
Fix tests
mcmire 843a5f5
Tweak wording in constraints file
mcmire d221d80
Merge branch 'main' into monorepo-poc
mcmire b3eeefd
Restore .editorconfig
mcmire d2c700e
Restore .github/workflows/build-lint-test.yml
mcmire e72cb39
Merge branch 'main' into monorepo-poc
mcmire d501396
Merge branch 'main' into monorepo-poc
mcmire 9742ca7
Restore isomorphic-fetch
mcmire 80f4ef7
Merge branch 'main' into monorepo-poc
mcmire d354a5f
Remove this instance of Yarn allow-scripts plugin
mcmire 296cced
Remove the Yarn version plugin
mcmire 4271859
Add a release.config.json for action-publish-release to read
mcmire 8147c62
Merge branch 'main' into monorepo-poc
mcmire 0b7becf
Bring back isomorphic-fetch type stub
mcmire dd828dd
Don't need this
mcmire b567f43
Tweak ordering to make diff easier to read
mcmire 65961a6
Add back some of the release steps to the README
mcmire 32286dd
Forgot about this file in a previous change
mcmire 8a2819e
Remove jest-it-up, simplify 'test' and 'test:watch' scripts
mcmire 4be5e3a
Restore previous changes to yarn.lock
mcmire 4c6ed67
Fix test failures
mcmire e792a1e
Use @types/uuid ^8.3.0
mcmire b23a926
Restore more dependency specifications
mcmire e04c6ec
Add a Yarn constraint to ensure that babel-runtime is added if eth-qu…
mcmire 3916201
Switch the Yarn constraint around so that the least version range win…
mcmire 0b38110
Make sure we ask people to submit a PR
mcmire 5589099
Revert check for undefined argument to hexToBN
mcmire 6b35da7
Clean up jest.config.packages.js a bit
mcmire 25a7afa
Assign a return type to setTimeout that works in both Node and browse…
mcmire 655d7a0
Fix these too
mcmire 41b1c56
I'm not being very thorough today, apparently
mcmire 113c7a0
Attempting to fix tests
mcmire ac20a31
Make this variable name more readable
mcmire 0476bfb
Update publish-release workflow to support new release process
mcmire 6c63a2a
Remove create-release-pr action
mcmire d1db12d
Switch rpc method tests back to using toStrictEqual
Gudahtt 6959246
add create-release-branch
Gudahtt ee323bd
COLLECTIBLE -> NFT
mcmire b6d186d
PermissionController: Link to Arch document from README, fix name in …
mcmire 5286350
Merge remote-tracking branch 'origin/main' into monorepo-poc
Gudahtt 2e36aa9
Fix README lint error
Gudahtt 839944e
Merge branch 'main' into monorepo-poc
mcmire 77a4b24
Make links to main README in per-package READMEs absolute
mcmire f3770ba
Merge branch 'main' into monorepo-poc
mcmire a08ab7e
Fix changelog validation on release branches
mcmire 3de137d
Bump action-publish-release
mcmire 4a0d96c
Fix publish-release workflow to restore compiled JS
mcmire 3c824fd
Update to 'action-npm-publish@v2
Gudahtt 6326c50
Merge remote-tracking branch 'origin/main' into monorepo-poc
Gudahtt cc9bdbb
Update workflows
mcmire 6c2cad8
Clean up publish-release workflow
mcmire 75cfcb4
Fix main workflow
mcmire 407a17f
Run tests in parallel
mcmire e43457a
Clean up workflows still
mcmire b65b4ae
Fix workflow
mcmire eb2670e
Run more workflow jobs in parallel to match the module template
mcmire 259f7b1
Fix workflow
mcmire 40a3e1e
Fix workflow
mcmire 743f434
Change action-publish-release back to v2
mcmire c9f960a
Remove --maxWorkers=1 from 'yarn test'
mcmire 853e460
Add back all-jobs-pass
mcmire 31cf514
Test on Node 18.x and 19.x
mcmire 43ca97e
Fix JSDoc for ContactEntry interface
mcmire 1b24bc7
Remove testEnvironmentOptions override and custom arrayMerge option i…
mcmire 4714429
Revert "Test on Node 18.x and 19.x"
mcmire 750ac80
Simplify doc script, add doc:clean
mcmire 0691206
Use ES2017 instead of ES2020
mcmire ca16296
Fix lint
mcmire a015279
Remove doc:clean
mcmire 3bf6120
Better align scripts to module template
mcmire 3a3b749
Fix lint
mcmire 0b957f8
Rename doc to build:docs for each package too
mcmire b7c1fbe
No need to install node for npm-publish jobs
mcmire File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
There are no files selected for viewing
This file was deleted.
Oops, something went wrong.
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,51 @@ | ||
| name: Main | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main] | ||
| pull_request: | ||
|
|
||
| jobs: | ||
| check-workflows: | ||
| name: Check workflows | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - name: Download actionlint | ||
| id: download-actionlint | ||
| run: bash <(curl https://raw.githubusercontent.com/rhysd/actionlint/7fdc9630cc360ea1a469eed64ac6d78caeda1234/scripts/download-actionlint.bash) 1.6.22 | ||
| shell: bash | ||
| - name: Check workflow files | ||
| run: ${{ steps.download-actionlint.outputs.executable }} -color | ||
| shell: bash | ||
|
|
||
| lint-build-test: | ||
| name: Lint, build, and test | ||
| needs: check-workflows | ||
| uses: ./.github/workflows/lint-build-test.yml | ||
|
|
||
| is-release: | ||
| name: Determine whether this is a release merge commit | ||
| needs: lint-build-test | ||
| if: startsWith(github.event.commits[0].author.name, 'github-actions') | ||
| runs-on: ubuntu-latest | ||
| outputs: | ||
| IS_RELEASE: ${{ steps.is-release.outputs.IS_RELEASE }} | ||
| steps: | ||
| - uses: MetaMask/action-is-release@v1 | ||
| id: is-release | ||
|
|
||
| publish-release: | ||
| name: Publish release | ||
| needs: is-release | ||
| if: needs.is-release.outputs.IS_RELEASE == 'true' | ||
| permissions: | ||
| contents: write | ||
| uses: ./.github/workflows/publish-release.yml | ||
|
|
||
| all-jobs-pass: | ||
| name: All jobs pass | ||
| runs-on: ubuntu-latest | ||
| needs: lint-build-test | ||
| steps: | ||
| - run: echo "Great success!" |
This file contains hidden or 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
This file contains hidden or 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -24,3 +24,6 @@ docs | |
| !.yarn/releases | ||
| !.yarn/sdks | ||
| !.yarn/versions | ||
|
|
||
| # typescript | ||
| *.tsbuildinfo | ||
Large diffs are not rendered by default.
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.