Skip to content

add usePantry().find()#37

Merged
mxcl merged 1 commit intomainfrom
find
Aug 2, 2023
Merged

add usePantry().find()#37
mxcl merged 1 commit intomainfrom
find

Conversation

@mxcl
Copy link
Copy Markdown
Contributor

@mxcl mxcl commented Aug 1, 2023

Refs #33

@mxcl
Copy link
Copy Markdown
Contributor Author

mxcl commented Aug 1, 2023

@jhheider does this accomplish what you wanted?

@jhheider
Copy link
Copy Markdown
Contributor

jhheider commented Aug 1, 2023

That's pretty good, except it requires a yaml read, even if the project name is found. A better solution probably checks the names on a pass, then does a pass reading yamls to check for provides/display-names. So you don't have to open up to over 600 files unless necessary.

@mxcl
Copy link
Copy Markdown
Contributor Author

mxcl commented Aug 1, 2023

The method returns an array, because it should—there might be conflicts.

I could add one that returns the first and then parallelizes a bit, need a use case where it's ok to do that tho.

Even tea/cli now errors if there are multiple results for a provides search

@mxcl mxcl merged commit 79f56e5 into main Aug 2, 2023
@mxcl mxcl deleted the find branch August 2, 2023 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants