chore(deps): remove unused cors dependency - #131
Conversation
saidsef
added
size/XS
dependencies
Up to standards ✅🟢 Issues
|
| Metric | Results |
|---|---|
| Complexity | 0 |
| Duplication | 0 |
NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
|
Application node-webserver-75f9f62-131-pr-reviews is in state Running. |
|
Application node-webserver-75f9f62-131-pr-reviews has been created. |
|
Application node-webserver-75f9f62-131-pr-reviews is now running new version of deployments manifests. |
The cors package was declared in app/package.json but is never imported or used anywhere in the codebase (a repo-wide search returns only the manifest line). Removing it drops cors and its transitive dependencies from the production install and the Docker image. Bump patch version 1.5.1 -> 1.5.2.
75f9f62 to
5ec36b4
Compare
saidsef
changed the title
|
Application node-webserver-75f9f62-131-pr-reviews has been created. |
|
Application node-webserver-75f9f62-131-pr-reviews has been deleted. |
|
Application node-webserver-5ec36b4-131-pr-reviews is in state Running. |
|
Application node-webserver-5ec36b4-131-pr-reviews has been created. |
|
Application node-webserver-5ec36b4-131-pr-reviews is in state Running. |
|
Application node-webserver-5ec36b4-131-pr-reviews is now running new version of deployments manifests. |
|
Application node-webserver-5ec36b4-131-pr-reviews has been deleted. |
1 participant
Summary
Removes the
corspackage fromapp/package.json. It was declared as a dependency but is never imported or used anywhere in the codebase - a repo-wide search returns only the manifest line. Removing it dropscorsand its transitive dependencies from the production install and the Docker image, with no change in behaviour.Also bumps the patch version
1.5.1->1.5.2.Changes
app/package.json: remove the unusedcorsdependency, bump version to 1.5.2Testing
Verified locally:
npm install --omit=devresolves cleanly;corsis absent fromnode_modulesServer is running on port 8080GET /healthzreturns{"status":"healthy"}(200)GET /returns the expected JSON (Hello World!, random number, 40-char sha1) (200)No functional changes.