Add monorepo support to project #682
Merged
Conversation
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
|
@danditomaso is attempting to deploy a commit to the Meshtastic Team on Vercel. A member of the Team first needs to authorize it. |
There was a problem hiding this comment.
Pull Request Overview
This PR restructures the repository into a monorepo by moving the existing web app under packages/web, adding workspace configurations for Deno, updating package scripts, and adjusting the pre-commit hook.
- Migrated all web files into
packages/web - Introduced package-level and root-level
deno.jsonworkspace configs - Updated
package.jsonscripts and dependencies, and adjusted the pre-commit hook to target the new structure
Reviewed Changes
Copilot reviewed 6 out of 458 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| packages/web/src/components/PageComponents/Connect/Serial.tsx | Removed an unused SerialPort type import |
| packages/web/src/components/LanguageSwitcher.tsx | Adjusted dropdown width from w-64 to w-48 |
| packages/web/package.json | Added generate:routes script, updated package script, moved/updated dependencies and removed lint-staged |
| packages/web/deno.json | Created a package-level Deno config with import aliases and TS options |
| deno.json | Configured root workspace to include packages/web and cleared root imports |
| .githooks/_/pre-commit | Scoped lint & format hook to packages/web |
Comments suppressed due to low confidence (1)
deno.json:5
- [nitpick] The root
deno.jsonnow has an emptyimportsmap after moving configs to package-level; if other packages rely on import remapping, ensure this change doesn't break their imports or consider consolidating common aliases at the root.
"imports": {},
|
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
Description
This pull request introduces several large scale changes to the repositry. As a result of the addition of several new applications to the web repo, we needed a way to orgnize simlar apps that will be sharing code and thus the monorepo. All files have been moved into /packages/web in a like for like migration so anything that worked previously will now live under /packages/web.
Changes Made
Checklist
CONTRIBUTING_I18N_DEVELOPER_GUIDE.md for more details)