-
-
Notifications
You must be signed in to change notification settings - Fork 13
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
[DEV] Bump to 1.2.0b2 #67
Conversation
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 ( |
@conda-forge-admin, please rerender |
…nda-forge-pinning 2022.07.08.02.15.19
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks right to me! Thanks @maresb!
Hmm, it looks like poetry-feedstock/recipe/meta.yaml Line 29 in 0dc31c1
Such circular dependencies are allowed on pypi but not in conda packages. The other problem is that I don't know how well it will work for a non-dev package ( |
Yes, this is the same circular dependency I was complaining about in #65. What do you mean by a "non-dev" package? At least the label for Maybe I should complain upstream about the circular dependency. |
It looks like the build is failing due to some missing packages. |
I noticed after the fact that |
Yes, the missing dependencies need to be built (or the recipes added) but the one that has me worried is:
I don't see any way to fix this unless |
Hmm, good point. These circular dependencies are very confusing. Conda tries to set up an environment for building Poetry, installs |
WIP for missing platformdirs: |
platformdirs is complete |
The remaining dependencies seem to now be in order. The circular dependency is still problematic. Hopefully something will happen with python-poetry/poetry#5980 |
closed via #71 |
Upstream diff
Checklist
0
(if the version changed)conda-smithy
(Use the phrase@conda-forge-admin, please rerender
in a comment in this PR for automated rerendering)