Skip to content

RFC: Only Registry Dependencies #593

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.

Sign up for GitHub

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

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

RFC: Only Registry Dependencies #593

Changes from 1 commit
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
12fe877
first draft
thescientist13 May 25, 2022
2559057
first round PR and meeting notes feedback
thescientist13 Jun 20, 2022
1434771
apply feedback to delegate the underlying implementation to be based …
thescientist13 Jul 31, 2022
fa5cf74
refine flag name from only-non-remote-deps to only-registry-deps
thescientist13 Jul 31, 2022
33dad50
rename RFC filename
thescientist13 Jul 31, 2022
d361131
refine npm query example per PR feedback
thescientist13 Aug 15, 2022
096f872
notes and bikeshedding clean up post ratification
thescientist13 Aug 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refine npm query example per PR feedback
  • Loading branch information
@thescientist13
thescientist13 committed Aug 15, 2022
commit d361131e73c8fb364a35fe8c385b83ff7ab74509
2 changes: 1 addition & 1 deletion accepted/0000-registry-only-dependencies.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -60,7 +60,7 @@ found N vulnerabilities

From a CLI perspective, this could easily (ideally) be delegated to the `npm query` command.
```sh
$ npm query ":root > *:is(:type(git,remote))"
$ npm query ":is(:type(git,remote))"
```

### Options
Expand Down