-
-
Notifications
You must be signed in to change notification settings - Fork 217
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
'six' package dependency removed #371
'six' package dependency removed #371
Conversation
from dbbackup.db.postgresql import (PgDumpBinaryConnector, PgDumpConnector, | ||
PgDumpGisConnector) |
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.
Do you think you could keep the format as before as that is easier to read?
from dbbackup.db.postgresql import (PgDumpBinaryConnector, PgDumpConnector, | |
PgDumpGisConnector) | |
from dbbackup.db.postgresql import ( | |
PgDumpBinaryConnector, | |
PgDumpConnector, | |
PgDumpGisConnector | |
) |
dbbackup/utils.py
Outdated
from __future__ import (absolute_import, division, print_function, | ||
unicode_literals) |
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.
Same here.
from __future__ import (absolute_import, division, print_function, | |
unicode_literals) | |
from __future__ import ( | |
absolute_import, | |
division, | |
print_function, | |
unicode_literals | |
) |
dbbackup/tests/test_utils.py
Outdated
callable_for_filename_template, | ||
DEV_NULL, add_public_gpg) | ||
clean_gpg_keys) |
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.
And change the formatting here as well.
Thanks a lot for making these changes! |
Thanks for the suggestion and will make the changes asap. BTW, I used the isort package to sort the imports. IMHO, we should add some kind of config file so that we can keep unified formatting rules across the repository/package. And later, the pipeline will take care of the checks |
I hope everything is fine now, isn't it? |
Thanks a lot @jerinpetergeorge |
Type of PR (feature, enhancement, bug fix, etc.)
enhancement
Description
Removed six package dependency and cleaned the codebase.
Fixes #370
Checklist