-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
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
feat: Add trogon to conda-forge #23391
Conversation
Hi! This is the friendly automated conda-forge-linting service. I wanted to let you know that I linted all conda-recipes in your PR ( Here's what I've got... For recipes/trogon:
|
This is a follow-up of Textualize/trogon#49 where hopefully we'd get a |
Hi! This is the friendly automated conda-forge-linting service. I just wanted to let you know that I linted all conda-recipes in your PR ( |
Also needed for conda-forge/mapyde-feedstock#14. |
hi @conda-forge/help-python -- can you please assist with this? There is no current LICENSE file but this is holding up another upstream project I already have in conda-forge. |
We need all the maintainers to comment confirming they are willing to be listed. PS: The license file is also a blocker. We need that to be able to re-package. |
Is there any suggestions on what i can do here? The problem is that the fact that this recipe is missing is making it harder for us to maintain the downstream packages on conda-forge. The corresponding issue is still open and unresolved (Textualize/trogon#49) which is unfortunate. And the maintainers don't seem to be responding as much, even though I've tried my best to reach out. |
Maybe they are too busy don't have the bandwidth to maintain this recipe. You can remove them and add a recipe with just yourself.
Indeed but a library without a license cannot be packaged. I would not even use it b/c the mention of the MIT in some places there imply in the existence of a license file... that is not there. |
Hi @ocefpaf - good news! An MIT |
For now, I'll submit an initial recipe that's from GitHub with the intention of updating it once a new release exists. |
Hi @conda-forge/help-python - this is ready to go! I opted to set the version as a dev release to indicate that the version from GitHub tarball that has the LICENSE is close to 0.5.1 rather than 0.5.0 (to be faithful to anyone expecting 0.5.0 on conda).
|
This reverts commit 9e1be4f.
This reverts commit 5229586.
Checklist
url
) rather than a repo (e.g.git_url
) is used in your recipe (see here for more details).