diff options
| author | Nathan Whitaker <17734409+nathanwhit@users.noreply.github.com> | 2024-09-06 10:18:13 -0700 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2024-09-06 17:18:13 +0000 |
| commit | 51f5f5789b3b00b327ee7130dc259d24ee631851 (patch) | |
| tree | ab6975ae1a49a3c036e99950947ec987e6d64edd /tests/specs/remove/basic | |
| parent | f0a3d206422af3177e0f36ed22802c1ccc6f7654 (diff) | |
feat(add): Add npm packages to package.json if present (#25477)
Closes https://github.com/denoland/deno/issues/25321
Ended up being a larger refactoring, since we're now juggling
(potentially) two config files in the same `add`, instead of choosing
one. I don't love the shape of the code, but I think it's good enough
Some smaller side improvements:
- `deno remove` supports `jsonc`
- `deno install --dev` will be a really simple change
- if `deno remove` removes the last import/dependency in the
`imports`/`dependencies`/`devDependencies` field, it removes the field
instead of leaving an empty object
Diffstat (limited to 'tests/specs/remove/basic')
| -rw-r--r-- | tests/specs/remove/basic/__test__.jsonc | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/tests/specs/remove/basic/__test__.jsonc b/tests/specs/remove/basic/__test__.jsonc index 2f4d82c88..495496b5c 100644 --- a/tests/specs/remove/basic/__test__.jsonc +++ b/tests/specs/remove/basic/__test__.jsonc @@ -12,5 +12,8 @@ }, { "args": ["eval", "console.log(Deno.readTextFileSync('deno.lock').trim())"], "output": "remove_lock.out" + }, { + "args": ["eval", "console.log(Deno.readTextFileSync('deno.json').trim())"], + "output": "{\n}\n" }] } |
