-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Switch to a pnpm-based monorepo with turbo #2677
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
base: main
Are you sure you want to change the base?
Conversation
🦋 Changeset detectedLatest commit: 7c5d02d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
This pull request is quite large, with changes spanning across 101 files and involving over 21,000 lines added and 48,000 lines removed. While the migration to pnpm seems to be a cohesive task, the updates to localization files might be unrelated to the core migration changes. To facilitate easier review and ensure that each set of changes receives the attention it deserves, I suggest splitting the localization updates into a separate pull request. This will help reviewers focus on the migration changes and the localization updates independently. Please consider creating a separate pull request for the localization updates. Thank you! |
Cool, with this we can share code easier and save node_module space :< |
I have worked some more on #2644, command line tool now uses monorepo structure |
@cte Any updates? We can start extracting the core modules that are unrelated to VSCode if this PR gets merged. |
…ooVetGit#2677) * initial * Menu * small clarity change * cleanup * Tests * Update .changeset/ninety-pots-rescue.md Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * Update src/test/TerminalCommandApprovalSettings.test.ts Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> * indentation * smooth transition * cleanup * Consolidated enabledActionsList for display, fixed unchecking issue * Updated shouldAutoApproveTool to better manage sub-options for future auto approve changes * removed test * cleanup * comments and extra bool check * cleanup * change to compare ids isntead of label strings * test in progress * test in progress * no tests for now * type safety change * cleanup * rename * comments * Copy --------- Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com> Co-authored-by: Saoud Rizwan <7799382+saoudrizwan@users.noreply.github.com>
Context
This allows us to carve our code up into separate packages for easy re-use across a variety of applications (VSCode extension, JetBrains extension, cli, evals, etc).
Implementation
Screenshots
How to Test
Get in Touch
Important
Switches project to a pnpm-based monorepo with turbo, updating package management, build processes, and configurations.
pnpm-workspace.yaml
to define package structure.package.json
scripts to usepnpm
andturbo
.changeset-release.yml
,code-qa.yml
).esbuild.js
andgenerate-types.mts
to align with pnpm..npmrc
files andflake
files.jest.config.js
andturbo.json
for new setup.src/i18n/locales
to reflect pnpm usage..gitignore
inwebview-ui
to ignore pnpm-related files.index.html
andindex.tsx
inwebview-ui
for new module paths.This description was created by
for 7c5d02d. It will automatically update as commits are pushed.