We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Proposition: I propose to strip whitespaces in BaseProcessor and assume whitespaced values as None.
Reason: Cells with whitespaces in large files are hard to find for users and displayed error is not clear to them.
Expected behavior:
def test_integer_processor_with_whitespace(): processor = IntegerProcessor() assert processor(' ') is None
Current result:
import_me.exceptions.ColumnError: is not an integer.
The text was updated successfully, but these errors were encountered:
Strip whitespaces in BaseProcessor. Issue best-doctor#34
3b3e2fb
Merge pull request #35 from LerikP/master
616207a
Strip whitespaces in BaseProcessor. Issue #34
No branches or pull requests
Proposition:
I propose to strip whitespaces in BaseProcessor and assume whitespaced values as None.
Reason:
Cells with whitespaces in large files are hard to find for users and displayed error is not clear to them.
Expected behavior:
Current result:
The text was updated successfully, but these errors were encountered: