Skip to content

Conversation

@Pixel998
Copy link
Contributor

Prerequisites checklist

What is the purpose of this pull request?

Adds Knip to check for unused exports, unused dependencies, and missing dependencies

What changes did you make? (Give an overview)

  • Added knip.json config and a root lint:unused script.
  • Added a "Lint Files, Dependencies, & Exports" step to our CI.
  • Removed unused BASELINE_FALSE export

Related Issues

eslint/js#695

Is there anything you'd like reviewers to focus on?

@Pixel998 Pixel998 requested a review from a team as a code owner January 29, 2026 23:55
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jan 29, 2026
@github-actions
Copy link
Contributor

github-actions bot commented Feb 9, 2026

Hi everyone, it looks like we lost track of this pull request. Please review and see what the next steps are. This pull request will auto-close in 7 days without an update.

@github-actions github-actions bot added the Stale label Feb 9, 2026
@@ -0,0 +1,4 @@
{
"$schema": "https://unpkg.com/knip@5/schema.json",
"entry": ["tests/**/*.{cts,js,ts}"]

Choose a reason for hiding this comment

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

Knip does not seem to recognize the glob pattern in the package.json test script.
The knip documentation advices to use --production instead of ignoring the test files.

@DMartens DMartens moved this from Needs Triage to Implementing in Triage Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Implementing

Development

Successfully merging this pull request may close these issues.

2 participants