Open
Conversation
Removing example that isn't needed
Contributor
|
🚀 Website Preview Deployed! Preview your changes at: https://cg-pr-663.billy-daly.workers.dev This preview will be automatically deleted when the PR is closed. |
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
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.
Summary
Changes proposed
~1.5.0/~0.75.0to~1.9.0/~0.79.0in both theexpress-jsandquickstarttemplates@eslint/js,@types/node,typescript-eslint,@vitest/eslint-plugin) to latest patch/minor versions@common-grants/corefrom^0.2.3to^0.2.4in theexpress-jstemplatevite,defu,rollup,qs,tar,yaml,ajv,brace-expansion)express-jstemplate frompackage-lock.json/.eslintrc.jstopnpm-lock.yaml/eslint.config.jspnpm-lock.yamlto thequickstarttemplate scaffold intemplate.jsonpackageManagerfield topnpm@10.18.3in both templatesexpress-js,quickstart,fast-api, and the Python examplesContext for reviewers
These are routine dependency updates to address security advisories in transitive dependencies. The pnpm overrides force patched versions of packages with known CVEs (e.g.
vite,qs,tar,rollup,yaml). The migration frompackage-lock.jsontopnpm-lock.yamlin the express-js template aligns with the workspace's use of pnpm.Verification: lock files were regenerated after applying the overrides and dependency bumps. The vite fix required removing an extraneous plugin and patching the config directly.
Additional information
There is a small change where folks will have to run
pnpm installafter runningtsp initThis change has been documented within a readme.