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

WIP: support for minimal JSON API #52

Open
wants to merge 20 commits into
base: main
Choose a base branch
from

Conversation

cmaumet
Copy link
Member

@cmaumet cmaumet commented Sep 13, 2018

No description provided.

@pep8speaks
Copy link

pep8speaks commented Sep 13, 2018

Hello @cmaumet! Thanks for updating the PR.

Line 40:80: E501 line too long (81 > 79 characters)
Line 240:80: E501 line too long (80 > 79 characters)
Line 473:80: E501 line too long (83 > 79 characters)
Line 503:80: E501 line too long (83 > 79 characters)
Line 505:80: E501 line too long (83 > 79 characters)
Line 544:80: E501 line too long (82 > 79 characters)
Line 554:80: E501 line too long (82 > 79 characters)
Line 555:80: E501 line too long (82 > 79 characters)
Line 603:80: E501 line too long (82 > 79 characters)
Line 622:80: E501 line too long (80 > 79 characters)
Line 657:80: E501 line too long (82 > 79 characters)
Line 675:80: E501 line too long (83 > 79 characters)
Line 717:48: E128 continuation line under-indented for visual indent
Line 724:80: E501 line too long (82 > 79 characters)
Line 734:80: E501 line too long (83 > 79 characters)
Line 737:80: E501 line too long (83 > 79 characters)
Line 740:80: E501 line too long (83 > 79 characters)
Line 742:80: E501 line too long (83 > 79 characters)
Line 746:80: E501 line too long (80 > 79 characters)
Line 747:80: E501 line too long (83 > 79 characters)
Line 864:80: E501 line too long (83 > 79 characters)
Line 874:80: E501 line too long (80 > 79 characters)
Line 886:80: E501 line too long (83 > 79 characters)
Line 917:80: E501 line too long (82 > 79 characters)
Line 918:80: E501 line too long (83 > 79 characters)
Line 924:80: E501 line too long (80 > 79 characters)
Line 925:80: E501 line too long (83 > 79 characters)
Line 928:80: E501 line too long (83 > 79 characters)
Line 965:80: E501 line too long (83 > 79 characters)
Line 980:80: E501 line too long (83 > 79 characters)
Line 987:80: E501 line too long (98 > 79 characters)
Line 1009:80: E501 line too long (98 > 79 characters)
Line 1019:80: E501 line too long (83 > 79 characters)
Line 1025:80: E501 line too long (82 > 79 characters)
Line 1049:80: E501 line too long (80 > 79 characters)
Line 1079:80: E501 line too long (81 > 79 characters)
Line 1080:80: E501 line too long (82 > 79 characters)
Line 1103:80: E501 line too long (82 > 79 characters)
Line 1129:80: E501 line too long (82 > 79 characters)
Line 1130:80: E501 line too long (83 > 79 characters)
Line 1131:80: E501 line too long (83 > 79 characters)
Line 1138:80: E501 line too long (80 > 79 characters)

Line 49:80: E501 line too long (80 > 79 characters)
Line 52:27: E261 at least two spaces before inline comment
Line 55:18: E128 continuation line under-indented for visual indent
Line 211:27: E261 at least two spaces before inline comment
Line 213:63: W291 trailing whitespace
Line 214:13: E128 continuation line under-indented for visual indent
Line 216:18: E128 continuation line under-indented for visual indent
Line 353:70: W291 trailing whitespace
Line 416:80: E501 line too long (80 > 79 characters)
Line 417:27: E261 at least two spaces before inline comment
Line 419:63: W291 trailing whitespace
Line 420:13: E128 continuation line under-indented for visual indent
Line 422:70: W291 trailing whitespace
Line 578:63: W291 trailing whitespace
Line 579:13: E128 continuation line under-indented for visual indent
Line 581:69: W291 trailing whitespace

Line 141:1: W293 blank line contains whitespace
Line 212:80: E501 line too long (90 > 79 characters)
Line 213:80: E501 line too long (128 > 79 characters)
Line 214:80: E501 line too long (126 > 79 characters)
Line 215:80: E501 line too long (128 > 79 characters)
Line 216:80: E501 line too long (126 > 79 characters)
Line 217:80: E501 line too long (128 > 79 characters)
Line 218:80: E501 line too long (126 > 79 characters)
Line 219:80: E501 line too long (124 > 79 characters)
Line 221:80: E501 line too long (81 > 79 characters)
Line 227:10: W291 trailing whitespace
Line 231:1: W293 blank line contains whitespace

Line 59:13: E128 continuation line under-indented for visual indent
Line 60:13: E128 continuation line under-indented for visual indent
Line 120:80: E501 line too long (84 > 79 characters)
Line 209:80: E501 line too long (89 > 79 characters)
Line 221:63: W291 trailing whitespace
Line 222:13: E128 continuation line under-indented for visual indent
Line 407:60: W291 trailing whitespace
Line 539:80: E501 line too long (88 > 79 characters)
Line 541:60: W291 trailing whitespace
Line 649:80: E501 line too long (83 > 79 characters)
Line 676:5: E303 too many blank lines (2)
Line 829:80: E501 line too long (82 > 79 characters)
Line 837:80: E501 line too long (81 > 79 characters)
Line 898:80: E501 line too long (86 > 79 characters)
Line 901:1: W293 blank line contains whitespace
Line 1080:80: E501 line too long (97 > 79 characters)
Line 1081:80: E501 line too long (91 > 79 characters)
Line 1082:80: E501 line too long (95 > 79 characters)
Line 1083:80: E501 line too long (95 > 79 characters)
Line 1084:80: E501 line too long (109 > 79 characters)
Line 1085:80: E501 line too long (109 > 79 characters)
Line 1086:80: E501 line too long (86 > 79 characters)
Line 1087:80: E501 line too long (83 > 79 characters)
Line 1096:80: E501 line too long (84 > 79 characters)
Line 1097:80: E501 line too long (90 > 79 characters)
Line 1098:80: E501 line too long (88 > 79 characters)
Line 1099:80: E501 line too long (90 > 79 characters)
Line 1100:80: E501 line too long (94 > 79 characters)
Line 1101:80: E501 line too long (89 > 79 characters)
Line 1102:80: E501 line too long (106 > 79 characters)
Line 1103:80: E501 line too long (104 > 79 characters)
Line 1104:80: E501 line too long (105 > 79 characters)
Line 1106:80: E501 line too long (83 > 79 characters)

Line 213:80: E501 line too long (86 > 79 characters)
Line 564:18: E128 continuation line under-indented for visual indent
Line 683:80: E501 line too long (93 > 79 characters)
Line 771:80: E501 line too long (82 > 79 characters)
Line 818:67: W291 trailing whitespace
Line 819:17: E128 continuation line under-indented for visual indent
Line 823:1: W293 blank line contains whitespace
Line 918:63: W291 trailing whitespace
Line 919:13: E128 continuation line under-indented for visual indent
Line 921:1: W293 blank line contains whitespace
Line 1118:63: W291 trailing whitespace
Line 1119:13: E128 continuation line under-indented for visual indent
Line 1121:1: W293 blank line contains whitespace
Line 1125:5: E303 too many blank lines (2)

Comment last updated on September 13, 2018 at 19:20 Hours UTC

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.

2 participants