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
Functionality to allow admins to switch on/off the line endings conversion on import #2819
Functionality to allow admins to switch on/off the line endings conversion on import #2819
Conversation
@@ -12,6 +12,7 @@ | |||
import sys | |||
import tempfile | |||
import zipfile | |||
import cProfile |
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.
Can you remove these changes?
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.
These are now removed.
@@ -12,7 +12,6 @@ | |||
import sys | |||
import tempfile | |||
import zipfile | |||
import cProfile |
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.
But the other changes to this file seem to remain - the part that used the cProfile import.
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.
I've removed the changes. The initial commit did'nt carry these.
Is there any reasons this has not been merged. I am happy to comply with any suggestions. |
LGTM, but ping @martenson (the libraries guru). |
works well, thanks @zipho for the contribution and @jmchilton with @nsoranzo for the review |
Functionality to allow admins to switch off the line endings conversion on import.
This is a cleanup to PR #2803