Skip to content
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

stockbase for QAStrategy #3

Merged
merged 7 commits into from Nov 10, 2019
Merged

stockbase for QAStrategy #3

merged 7 commits into from Nov 10, 2019

Conversation

@yutiansut
Copy link
Member

yutiansut commented Nov 10, 2019

No description provided.

@pep8speaks

This comment has been minimized.

Copy link

pep8speaks commented Nov 10, 2019

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

Line 161:1: W293 blank line contains whitespace

Line 29:80: E501 line too long (123 > 79 characters)
Line 139:5: E303 too many blank lines (2)
Line 145:80: E501 line too long (100 > 79 characters)
Line 146:9: E265 block comment should start with '# '
Line 151:80: E501 line too long (91 > 79 characters)
Line 152:80: E501 line too long (117 > 79 characters)
Line 163:1: W293 blank line contains whitespace

Copy link

codelingo bot left a comment

LGTM

@yutiansut yutiansut merged commit c775339 into QUANTAXIS:master Nov 10, 2019
2 checks passed
2 checks passed
AccessLint Review complete
Review Action No issues found.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.