Skip to content
This repository was archived by the owner on Feb 2, 2024. It is now read-only.

Use PyArrow for reading CSV#270

Merged
shssf merged 4 commits intoIntelPython:masterfrom
PokhodenkoSA:arrow
Nov 18, 2019
Merged

Use PyArrow for reading CSV#270
shssf merged 4 commits intoIntelPython:masterfrom
PokhodenkoSA:arrow

Conversation

@PokhodenkoSA
Copy link
Copy Markdown
Contributor

No description provided.

@pep8speaks
Copy link
Copy Markdown

pep8speaks commented Nov 5, 2019

Hello @PokhodenkoSA! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 398:32: E231 missing whitespace after ','
Line 421:5: E722 do not use bare 'except'
Line 421:11: E701 multiple statements on one line (colon)
Line 465:22: E201 whitespace after '{'
Line 465:96: E202 whitespace before '}'
Line 467:22: E201 whitespace after '{'
Line 467:103: E202 whitespace before '}'
Line 475:5: E722 do not use bare 'except'
Line 475:11: E701 multiple statements on one line (colon)
Line 506:16: E222 multiple spaces after operator

Line 412:25: E128 continuation line under-indented for visual indent
Line 413:21: E124 closing bracket does not match visual indentation
Line 416:25: E128 continuation line under-indented for visual indent
Line 417:21: E124 closing bracket does not match visual indentation
Line 805:17: E128 continuation line under-indented for visual indent
Line 806:17: E128 continuation line under-indented for visual indent
Line 818:17: E128 continuation line under-indented for visual indent
Line 819:17: E128 continuation line under-indented for visual indent

Comment last updated at 2019-11-17 21:44:46 UTC

@shssf shssf merged commit 515f7b5 into IntelPython:master Nov 18, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants