-
Notifications
You must be signed in to change notification settings - Fork 38
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
[WIP] feat: dependents
and dependencies
qualifiers
#235
base: master
Are you sure you want to change the base?
Conversation
@larsgw One issue with dependants is that it can be really huge, for instance, there’s hundreds, probably thousands, packages that depend on lodash. That’s why I wanted to understand the use-case and it seems that you only need dependencies. Having said that, you need to set an array of dependency names based on the packageJson.dependencies & peerDepedencies, and index them on Elasticsearch. You must also to add that field to the elasticsearch config that lives in the /config folder. Once that is done, you just need to support a new modifier in |
Yeah, I'm mixing those up a bit. Let me get this straight: for searching dependents of a pkg we need
Thank you, I couldn't find that part.
How would that work in ES?
I was thinking of continuing the |
I think if we store an array of objects, we will need nesting: https://www.elastic.co/guide/en/elasticsearch/guide/current/nested-objects.html
You can choose what sounds best for you and we decide on the final name on the PR. |
Add dependencies and dependents to the ElasticSearch config.
I added |
anything we can do to help this along? 🤗 |
friendly reminder about this 🤗 I could start playing around myself to do an actual code contribution but that seems to almost exactly what is needed? so if there is any possibility to finish this one up would be nice 🤗 if not just let us know and we will try to prepare a PR ourself... 😅 |
Regarding the actual state of this PR, the Other than that, the PR is looking good but there's still some work to do (checklist). |
I'm a little confused with As outlined here npms-io/npms#29 (comment) the information that we are after is to find out which direct dependcies are used. e.g. so I can say "Show me all packages that have lit-element 2.x as a direct dependency" |
@daKmoR for that use-case, we only need |
uh yeah that sounds way better 🤗 maybe even call it so I am all for dropping @larsgw are still on board? or should I try my luck? PS: I probably need to do an |
I think removing |
WIP:
dependents
dependencies
Closes npms-io/npms#29