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

Check a repo's already installed packages #163

Open
xiniria opened this issue Nov 19, 2020 · 3 comments
Open

Check a repo's already installed packages #163

xiniria opened this issue Nov 19, 2020 · 3 comments
Assignees

Comments

@xiniria
Copy link
Contributor

xiniria commented Nov 19, 2020

Expected Behavior

I think a good feature of npq would be to check if the repo's already installed packages have any issues, in the same way that it checks if a package has any issues when we want to install it. It would be a great improvement and could help set up a CI check for installed packages!

Possible Solution

Perhaps a new command npq check that detects the package.json in the current directory and runs the marshalls on all found packages in dependencies and devDependencies and reports any issues; here of course it wouldn't ask for confirmation before installing since the packages are already installed.

@lirantal
Copy link
Owner

Yep, that's a great idea!
As simple as just traversing the direct packages, right? otherwise we might be needing to go through hundreds or thousands of them and it'll be slow and also perhaps not very actionable if you don't have a way to fix them.

Would you like to work on this? I'd be happy to merge it in

@xiniria
Copy link
Contributor Author

xiniria commented Nov 20, 2020

As simple as just traversing the direct packages, right? otherwise we might be needing to go through hundreds or thousands of them and it'll be slow and also perhaps not very actionable if you don't have a way to fix them.

Yes of course, just the top-level packages.

Would you like to work on this? I'd be happy to merge it in

I have a bit of work for now but I think I can start working on it in 2 or 3 weeks. Thanks!

@lirantal
Copy link
Owner

Sure, no rush! thanks for suggesting. Take the time you need, these are challenging time.
Stay safe and healthy ❤️

@xiniria xiniria mentioned this issue Nov 26, 2020
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants