-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Update conda-forge feedstock to include missing dependencies #1279
Comments
Yes, the maintainers were also quite slow with the last PRs updating the version: Sadly I cannot merge them, or is there I way I can take care of that? |
I guess, it's best to update the maintainers' list. I have asked to add myself to it. And I personally feel that it is always a good strategy to include one of the maintainers of the original package as a maintainer of the corresponding conda-forge feedstock. |
@nreimers The PR has been merged and I was added to the list of maintainers for conda-forge/sentence-transformers-feedstock. |
Great, thank you so much for this :) |
@tomaarsen I think this PR could be closed as well. |
I have already pushed a PR β against this issue π to https://github.com/conda-forge/sentence-transformers-feedstock. It has been almost a week now, but the maintainers have not responded yet.
All that is necessary is to accept and merge the PR (which passed all the CIs).
cc: @ndmaxar, @oblute, @rluria14
The text was updated successfully, but these errors were encountered: