-
Notifications
You must be signed in to change notification settings - Fork 31
Transition to Yarn Workspaces #459
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
Draft
leeyi45
wants to merge
139
commits into
master
Choose a base branch
from
yarn-workspaces
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+201,131
−186,325
Draft
Changes from all commits
Commits
Show all changes
139 commits
Select commit
Hold shift + click to select a range
2d223dc
Rearrange everything into its own git workspace
leeyi45 fe8853b
Some more tidying up
leeyi45 8ad4331
Refactor all bundles and tabs
leeyi45 e6cfbfa
Add moduleslib
leeyi45 159790b
Add lint plugin
leeyi45 b59f345
modules
leeyi45 5275cc0
Added working commands
leeyi45 aa889e6
Added templates
leeyi45 754a1de
Add more buildtools
leeyi45 fad4626
Further work on buildtools
leeyi45 0274c12
Working typedoc building
leeyi45 58abebd
More work
leeyi45 ea46cb7
A working set of commands
leeyi45 36958df
Add new commands for tabs and potentially jsdoc for bundles
leeyi45 be9c95a
Add manifests
leeyi45 ff1941c
Move stuff to a separate lib folder
leeyi45 0c23c29
fix lintplugin things
leeyi45 4de476e
Some working tests
leeyi45 5c6db3d
Use defineTab
leeyi45 e2b1043
Add package json to devserver
leeyi45 f66eeaf
Migrate to vitest
leeyi45 32b4c5f
Migrate to vitest
leeyi45 765b4e6
Relocate things
leeyi45 0ce0e8e
Update test files and add csg patch
leeyi45 c6d3970
Update root repository items
leeyi45 1f18c3d
Update eslint config to include configuration names
leeyi45 96904f7
Update lintplugin
leeyi45 2fbf028
Update devserver vite configs
leeyi45 1793e9b
Update tab test configuration
leeyi45 a3a2c52
Finish creating output formatters and tests
leeyi45 54a43c5
Add typedoc option to tsconfigs
leeyi45 a75236e
Misc changes to root repo
leeyi45 bd91c5e
Remove commented out code that was used only for testing
leeyi45 d6c722f
Change devserver to resemble a properly installed package
leeyi45 35674d5
Update modules-lib and lintplugin packages
leeyi45 f345516
Update testing scripts and github workflow
leeyi45 4c1832e
Fix incorrect js-slang resolution
leeyi45 bbfccd4
Update incorrect package export fields
leeyi45 f63c9b4
Finish implementing build all command and tests
leeyi45 1b647bb
Move vitest config to JS files so that tsc and eslint stop complaining
leeyi45 0890958
Update bundle and tab vitest configurations
leeyi45 5f520ee
Fix testing configurations and update scripts
leeyi45 d06d11b
Update auxillary repo stuff
leeyi45 23655ff
Update dependencies to use versions from master branch
leeyi45 88d2a17
Merge commit '73beb14a7475300e90b06de599b9311d01228fce' into yarn-wor…
leeyi45 f530275
Add missing newlines
leeyi45 afc0f81
Fix broken manifest schema not working with VSCode
leeyi45 bb73309
Add json files to ESLint configuration to enforce eol-last
leeyi45 fea8adb
Remove yarn constraints check from CI
leeyi45 3c47ca0
Add skipLibCheck to ignore a type error in js-slang
leeyi45 064db71
Also add skipLibCheck to libraries
leeyi45 2b8f05f
Fix broken test commands
leeyi45 e3aa9b3
Fix tests that were erroring in the CI environment
leeyi45 c2a1c05
Increase timeout for build:all test
leeyi45 f2374e4
Circumvent the out of memory error for eslint
leeyi45 e680c3e
Fix the test command
leeyi45 aa9632f
Make github action workflows conditional
leeyi45 a09ebf6
Fix tsc not properly outputting javascript files
leeyi45 e8ab993
Add new tsc command that automatically filters tests
leeyi45 284d0a4
Fix missing parameter for runTsc
leeyi45 7ba95d7
Update test that's timing out
leeyi45 122c289
Update git hooks
leeyi45 6617ed7
Add prebuild all command
leeyi45 5bbe91b
Update husky git hook to use prebuild command
leeyi45 fa45c91
Fix broken scrabble tests
leeyi45 81d8a5c
Fix broken CI workflow
leeyi45 3e02635
Fix the still broken workflow
leeyi45 6dd3104
Re-add missing preinstall script
leeyi45 bdd1737
Make workflow build manifest if devserver is required
leeyi45 5e962c8
Update workflows
leeyi45 d4bc685
Add a vitepress documentation server for the repository
leeyi45 0c9152e
Add more documentation
leeyi45 1e832f4
Change vitepress docs to use vitepress-sidebar
leeyi45 0a5558a
Begin work on second docs section
leeyi45 4a6cfc2
Add markdown rules to linting
leeyi45 3ea31c4
Fix the sidebar ordering
leeyi45 13f19ae
Fix documentation not being correctly generated for modules-lib
leeyi45 a373b09
Fix broken tests and installs
leeyi45 312c0fe
Fix broken tsc test
leeyi45 25bc826
Change document configurations
leeyi45 7ea67db
Try to fix devserver tests failing
leeyi45 412b6b4
Continue to fix broken tests
leeyi45 6d15145
Add modules-lib as a dependency for devserver
leeyi45 ec6c043
Continue fixing broken references
leeyi45 a3cec8e
Fix a broken file name
leeyi45 621dfc1
Fix more broken tests and add documentation for repo tools
leeyi45 06cc233
Fix broken test
leeyi45 def7151
Change the defineTab import
leeyi45 b113ce3
Fix incorrect snapshots
leeyi45 d94bca0
Fix weirdness with WebGLCanvas import path name
leeyi45 20a475b
Please finally fix the issues with the tab exports
leeyi45 889b31b
guess it was still broken
leeyi45 9c6fea1
Trying to fix devserver tests this time
leeyi45 4ff050c
Update configuration for docserver
leeyi45 fda04fd
Incorporate JSON validation errors properly
leeyi45 4634304
Add ability to use mermaid diagrams in docserver
leeyi45 299c19c
Enable import sorting via eslint plugin
leeyi45 06102fb
More linting changes
leeyi45 ec98f60
Fix linting issues
leeyi45 f2a58b2
Fix broken buildtools test
leeyi45 74e22f4
Update slow tests
leeyi45 de89125
Fix linting
leeyi45 7da62e8
Update doc server build command
leeyi45 89c924e
Update broken github workflow
leeyi45 49305e5
Update linting and documentation
leeyi45 486be4b
Remove unnecessary expect error
leeyi45 f07f097
Add bundle conventions to docs
leeyi45 11f9d2b
Add way more information about bundle writing
leeyi45 49fdb9d
Update buildtools and docs
leeyi45 855abae
Changed how commands are executed to delegate executing over multiple…
leeyi45 9188068
Add markdown tree plugin
leeyi45 e7790bb
Update workflow and linting
leeyi45 9c02cca
Fix broken tests
leeyi45 5ce8c97
Redo modules-lib animations
leeyi45 c0dddea
Addes tests for modules-lib components
leeyi45 bfca65c
Fix dead link in docs
leeyi45 c113a0b
Fix some broken things
leeyi45 a71f041
Actually use the react plugin's rules
leeyi45 446bede
configure eslint to use react plugin rules
leeyi45 8a9c7f2
Change how tests are run to enable Tabs to make use of Vitest browser…
leeyi45 8ab2530
Add playwright to make sure that tabs tests can run in CI
leeyi45 8850616
Try to fix the workflow....
leeyi45 8f93c96
Update lockfile
leeyi45 9081c90
Update github workflow
leeyi45 8091ed3
Fix a 1 character mistake :(
leeyi45 87e7784
still trying to fix a test
leeyi45 ed05177
Add new lib-compiler package to replace repeated code for building li…
leeyi45 7427aa9
Add lib-compiler
leeyi45 d42ad0e
Add a #region comment rule
leeyi45 c69bbb7
Try very hard to fix vitest
leeyi45 55f35a5
Add missing script
leeyi45 370a9c0
Try and fix lib compiler not installing
leeyi45 16a7ade
Update docs
leeyi45 4caa3cc
please work
leeyi45 53c3210
Switch to using buildtools testing
leeyi45 b995491
Use root config as a shared configuration for vitest instead
leeyi45 87e71dc
Fix missing tests for lint plugin tab-type
leeyi45 50af717
Fix incorrect test:devserver command
leeyi45 4f574c0
Add tests for test command
leeyi45 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 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
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 was deleted.
Oops, something went wrong.
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Incorrect file permissions (0644), this hook won't run |
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 |
---|---|---|
@@ -1 +1,6 @@ | ||
yarn build --lint --tsc | ||
yarn constraints | ||
yarn workspaces foreach -ptR -j 5 --since --include "./src/{bundles,tabs}/*" run buildtools prebuild | ||
yarn workspaces foreach -ptR -j 5 --since --include "./src/{bundles,tabs}/*" run test | ||
yarn workspaces foreach -ptW --since --include "./{lib,devserver}/*" run lint | ||
yarn workspaces foreach -ptW --since --include "./{lib,devserver}/*" run tsc | ||
yarn workspaces foreach -ptW --since --include "./{lib,devserver}/*" run test |
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 |
---|---|---|
@@ -1 +1 @@ | ||
20.11.1 | ||
22.16.0 |
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,20 @@ | ||
{ | ||
"files.insertFinalNewline": true, | ||
"json.schemas": [ | ||
{ | ||
"fileMatch": ["src/bundles/**/manifest.json"], | ||
"url": "./lib/buildtools/src/build/modules/manifest.schema.json" | ||
}, | ||
{ | ||
"fileMatch": ["tsconfig.json", "tsconfig.*.json"], | ||
"schema": { | ||
"allOf": [{ "$ref": "http://json.schemastore.org/tsconfig" }], | ||
"properties": { | ||
"typedocOptions": { | ||
"allOf": [{ "$ref": "https://typedoc.org/schema.json" }] | ||
} | ||
} | ||
} | ||
} | ||
] | ||
} |
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.
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.
Hmm, possible to run some of these steps in parallel using multiple jobs/a matrix strategy?