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

Feature/test wmp #84

Merged
merged 16 commits into from Oct 9, 2021
Merged

Feature/test wmp #84

merged 16 commits into from Oct 9, 2021

Conversation

alshapton
Copy link
Owner

No description provided.

@pep8speaks
Copy link

pep8speaks commented Oct 9, 2021

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

Line 28:80: E501 line too long (99 > 79 characters)

Line 551:7: E111 indentation is not a multiple of four
Line 552:7: E111 indentation is not a multiple of four
Line 553:7: E111 indentation is not a multiple of four
Line 554:7: E111 indentation is not a multiple of four
Line 555:7: E111 indentation is not a multiple of four
Line 794:64: E702 multiple statements on one line (semicolon)
Line 794:73: E203 whitespace before ','
Line 810:9: E113 unexpected indentation

Line 8:80: E501 line too long (99 > 79 characters)

Line 56:80: E501 line too long (81 > 79 characters)
Line 57:80: E501 line too long (114 > 79 characters)

Line 32:80: E501 line too long (90 > 79 characters)
Line 36:80: E501 line too long (91 > 79 characters)
Line 40:80: E501 line too long (97 > 79 characters)
Line 44:80: E501 line too long (97 > 79 characters)

Comment last updated at 2021-10-09 20:29:34 UTC

@alshapton alshapton merged commit 5e9ea13 into develop Oct 9, 2021
@alshapton alshapton deleted the feature/test-wmp branch October 26, 2021 22:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants