-
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
remove top pin for jsonschema #51
Conversation
Hi! Could this be merged please? It causes issues with dbt-core |
Also curious if this can be merged. @dnascimento @bollwyvl did you guys find a workaround? i'm currently trying to install |
Yerp: as mentioned, on When this repo finally makes a canonical release with the fix, the conda-forge build will break when the bot PR finds it, we'll remove the patch, and forget this happened. |
hmm okay, that sounds good! Could you help me out how to use the conda-forge version in my installation? I've never worked with that. because of my CI/CD requirements I need to specify my dependencies in the
the dbt-core version uses
|
The reverse is not true, and tools like conda-lock can help bridge the gap between the two. If that's not an option, you could be to take the patches applied on the conda-forge repo, and build your own wheels. If that is incompatible with your workflow, and relying on "stuff you find on pypi" as being "the truth," is more important than "it works," then maybe that's a problem with your workflow. The owners of this repo know this is causing problems for end users: if you are a paying |
I was already thinking along that axis, it makes sense. We're using a private (AWS CodeArtifact) repository manager, that should work :)
I'll try to get in contact with Thanks! |
Already merged this change. Closing. |
Hi! Thanks for
hologram
!This removes the top pin for
jsonschema
. Under test, this passes with ~90% coverage over on conda-forge/hologram-feedstock#4 where this pin is becoming problematic for downstreams.See also #50.