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
Packaging several async python web libraries / databases connectors #76652
Conversation
I realize this is larger PR but as I was enabling the test suites of starlette/httpx/fastapi it required these dependencies. |
5bca988
to
c93eac6
Compare
… enable tests fastapi is the only dependent package of starlette. Downgrading version and enabling full tests to fix fastapi.
c93eac6
to
708d34a
Compare
I ran Result of 13 package were built:
|
Running Result of 1 package were built:
|
nix-review passes for me with same |
Much work was done in this PR to ensure that fastapi,starlette,pydantic and all new packages have all tests enabled. Previously simple tests were performed (e.g. imports work). Also moved all new/refactored derivations to use github (in my opinion after submitting many many PRs to nixpkgs repositories are better than pypi) one but not all of the reasons is that authors typically exclude tests from the pypi release.
All packages have full test suite enabled. Thus going forward we should have great certainty that they are not broken. starlette was reverted from 0.12.13 -> 0.12.9 since it's only dependent package
fastapi
pins on this exact version (it is broken with newer versions). Hopefully we can remove this pin soon.Motivation for this change
These are some great core async libraries to have available for web development.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @