Skip to content

SCANNPM-100 Add knip#216

Merged
zglicz merged 5 commits intomasterfrom
knip
Jul 24, 2025
Merged

SCANNPM-100 Add knip#216
zglicz merged 5 commits intomasterfrom
knip

Conversation

@zglicz
Copy link
Contributor

@zglicz zglicz commented Jul 24, 2025

SCANNPM-100

As part of adding it, we find that @types/mkdirp is not necessary as the package 'mkdirp' publishes the types itself.

@zglicz zglicz requested a review from vdiez July 24, 2025 09:46
@hashicorp-vault-sonar-prod hashicorp-vault-sonar-prod bot changed the title Add knip SCANNPM-100 Add knip Jul 24, 2025
@@ -1,42 +0,0 @@
/*
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we need this file as it's the API entrypoint

@sonarqubecloud
Copy link

🤖 Pull Request summary

Update project configuration and add dead code detection.

  • Added Knip integration: New CI task and configuration file for detecting unused dependencies and dead code
  • Dependency updates: Pinned exact versions by removing caret prefixes from multiple package dependencies
  • Configuration cleanup: Removed .pmgrc.toml file and added .npmrc with save-exact=true setting
  • Package optimization: Removed rimraf dev dependency and unused @types/mkdirp type definition

💬 Please send your feedback

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@zglicz zglicz merged commit 5110218 into master Jul 24, 2025
8 checks passed
@zglicz zglicz deleted the knip branch July 24, 2025 10:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants