cleanup: remove unused dev dependency #376

Merged
barrettruth merged 1 commit from issue-363-dev-dependency-ignores into main 2026-05-11 21:23:41 +00:00
Owner

Problem

The root package still listed react-grab as a dev dependency even though the repo no longer uses it. Root dist/ and CLI node_modules/ generated artifacts also relied on non-repo ignore coverage.

Solution

  • remove react-grab from package.json and pnpm-lock.yaml
  • add repo-local ignores for /dist/ and /cli/node_modules/
  • keep existing generated artifact ignores intact

Verification

  • pnpm install --lockfile-only
  • pnpm why react-grab
  • git check-ignore -v dist/example.js cli/node_modules/example cli/dist/example.js node_modules/example
  • nix develop --command just lint
  • nix develop --command just test
  • nix develop --command just build

Closes #363

Tracker: #355

## Problem The root package still listed `react-grab` as a dev dependency even though the repo no longer uses it. Root `dist/` and CLI `node_modules/` generated artifacts also relied on non-repo ignore coverage. ## Solution - remove `react-grab` from `package.json` and `pnpm-lock.yaml` - add repo-local ignores for `/dist/` and `/cli/node_modules/` - keep existing generated artifact ignores intact ## Verification - `pnpm install --lockfile-only` - `pnpm why react-grab` - `git check-ignore -v dist/example.js cli/node_modules/example cli/dist/example.js node_modules/example` - `nix develop --command just lint` - `nix develop --command just test` - `nix develop --command just build` Closes #363 Tracker: #355
cleanup: remove unused dev dependency
All checks were successful
quality / Test (pull_request) Successful in 16s
quality / Lint (pull_request) Successful in 20s
quality / Build (pull_request) Successful in 42s
6a37772ba6
barrettruth deleted branch issue-363-dev-dependency-ignores 2026-05-11 21:23:42 +00:00
Sign in to join this conversation.
No reviewers
No milestone
No project
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
barrettruth/delta!376
No description provided.