Skip to content
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

chore: add 'lint:deps' script to check for unused and unlisted deps #2477

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Commits on Oct 10, 2024

  1. chore: add 'lint:deps' script to check for unused and unlisted deps

    This uses knip (https://knip.dev) to check dependencies, in hopes of avoiding
    issues like open-telemetry#2473. This adds 'npm run lint:deps' and adds that to the
    existing 'npm run lint'.
    
    This change includes fixes for a handful of unused and unlisted deps.
    For now knip is configured to only check 'production' deps. Checking non-prod
    deps results in way too many false positives.
    trentm committed Oct 10, 2024
    Configuration menu
    Copy the full SHA
    3f236e7 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    47ee30b View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    82c2a8e View commit details
    Browse the repository at this point in the history
  4. Configuration menu
    Copy the full SHA
    722c7b4 View commit details
    Browse the repository at this point in the history

Commits on Oct 11, 2024

  1. running knip: explicit version, --yes to avoid prompt

    After reading more about how npx works
    (https://docs.npmjs.com/cli/v8/commands/npm-exec)
    I'm more confident that running knip via npx (to workaround the
    install-tree issue) is fine for dev and for CI.
    trentm committed Oct 11, 2024
    Configuration menu
    Copy the full SHA
    971f52e View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    793a566 View commit details
    Browse the repository at this point in the history