- Notifications
You must be signed in to change notification settings - Fork126
Relax panda and alembic requirements#116
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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
susodapop commentedMay 3, 2023
Good idea. I'm working on this exact change locally, actually. Just running our e2e tests before pushing my own branch. This PR won't quite work because you haven't included the DCO in your commit (see CONTRIBUTING.md for details). |
bkyryliuk commentedMay 3, 2023
that's great. TY! Will check on DCO |
Signed-off-by: Bogdan Kyryliuk <b.kyryliuk@gmail.com>
susodapop commentedMay 9, 2023
I've mirrored this change in#119 so I can verify it builds. This change will be part of v2.5.2 releasing shortly. |
Pandas 1.3+ has a regression:pandas-dev/pandas#42549
This PR proposes relaxing alembic and panda requirements