-
Notifications
You must be signed in to change notification settings - Fork 79
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
Added packages python-calamine and openpyxl #291
Conversation
Thanks Sujan! You are right the psycopg error can ignored 👍 |
for more information, see https://pre-commit.ci
Updates:
|
@spwoodcock it is good to merge, but a test with these changes would be appreciated. |
Will check and merge today! |
I updated to remove However I kept
|
I started writing a very basic test for this. @Sujanadh could you possibly finish this off? |
I made a few small changes! Also added some tests - let me know if this works / you agree with the edits & I will merge 👍 |
I tested it. it is good to go |
Updates
python-calamine
andxlsxwriter
are added as packages. These packages are required to be used as an engine with pandas for reading and writingxlsx
files.What I encountered?
Got this dependency error from psycopg2 , I don't think it is related with these packages, I guess we can ignore it?