-
-
Notifications
You must be signed in to change notification settings - Fork 512
chore: reduce local dependencies and update workflows #5863
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
Conversation
lerna-lite to CI@vscode/vsce and lerna-lite to CI
@vscode/vsce and lerna-lite to CI@vscode/vsce and lerna-lite in CI
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.
Pull request overview
This PR optimizes the development setup by moving @vscode/vsce and lerna-lite packages to CI-only installation, reducing local node_modules size from 313.1 MB to 202.8 MB. The packages are now installed globally in CI workflows using pnpm add -g when needed for publishing operations.
Key Changes:
- Removed
@vscode/vsceandlerna-litepackages from local devDependencies - Updated package scripts to use
npxfor extension packaging - Created new CI workflows for package and extension publishing
- Streamlined existing CI workflows with consistent Node.js version (24) and formatting
Reviewed changes
Copilot reviewed 13 out of 14 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| package.json | Removed @lerna-lite/cli and @lerna-lite/publish from devDependencies |
| extensions/vscode/package.json | Removed @vscode/vsce devDependency, updated pack script to use npx, removed release script |
| lerna.json | Removed command configurations, test-workspace from packages list, and extra config options |
| test-workspace/package.json | Removed version field |
| pnpm-lock.yaml | Removed all dependencies related to vsce and lerna-lite |
| .github/workflows/publish-packages.yml | New workflow for publishing packages with global lerna-lite installation |
| .github/workflows/publish-extension.yml | New workflow for publishing extensions with global vsce installation |
| .github/workflows/bump-version-comment.yml | New workflow for version bumping via PR comments |
| .github/workflows/*.yml | Updated Node.js version to 24, improved formatting and naming |
| .github/workflows/extension-release.yml | Deleted (replaced by publish-extension.yml) |
| .github/workflows/close-cannot-repoduce-issues.yml | Deleted |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@vscode/vsce and lerna-lite in CI
Only install and use
@vscode/vsceandlerna-litein CI to reduce localnode_modulessize. (313.1 MB -> 202.8 MB)Add
bump-version-comment.ymlworkflow: Trigger CI to update version numbers inpackage.jsonfiles when commenting@bump-patchor@bump-minor.Add
release.ymlworkflow: When pushing a commit with a message formatted asv[0-9]+\.[0-9]+\.[0-9]+, push tags in CI and publish to npm, Open VSX, and VSCode Marketplace.