1: D100 Docstring missing
2: L102 copyright year is outdated, expected 2015 but got 2014
23:1: I101 Imported names are in the wrong order. Should be cPickle, calendar, cgi, commands, datetime, os, random, time
23:16: E401 multiple imports on one line
25:1: I101 Imported names are in the wrong order. Should be CFG_BIBCIRCULATION_ILL_STATUS_CANCELLED, CFG_BIBCIRCULATION_ITEM_STATUS_CANCELLED, CFG_BIBCIRCULATION_ITEM_STATUS_CLAIMED, CFG_BIBCIRCULATION_ITEM_STATUS_IN_PROCESS, CFG_BIBCIRCULATION_ITEM_STATUS_NOT_ARRIVED, CFG_BIBCIRCULATION_ITEM_STATUS_ON_LOAN, CFG_BIBCIRCULATION_ITEM_STATUS_ON_ORDER, CFG_BIBCIRCULATION_ITEM_STATUS_ON_SHELF, CFG_BIBCIRCULATION_ITEM_STATUS_OPTIONAL, CFG_BIBCIRCULATION_REQUEST_STATUS_DONE, CFG_BINDIR, CFG_CERN_SITE, CFG_SITE_NAME, CFG_SITE_URL, CFG_TMPDIR
25:1: I201 Missing newline before sections or imports.
42:1: I101 Imported names are in the wrong order. Should be get_collection_reclist, get_most_popular_field_values, perform_request_search, search_pattern
42:1: I100 Imports statements are in the wrong order. from invenio.legacy.search_engine should be before from invenio.utils.url
46:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibrecord should be before from invenio.legacy.search_engine
50:1: I101 Imported names are in the wrong order. Should be book_information_from_MARC, book_title_from_MARC
50:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcirculation.utils should be before from invenio.legacy.websubmit.admin_dblayer
52:1: I101 Imported names are in the wrong order. Should be get_borrower_data, get_id_bibrec
52:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibcirculation.db_layer should be before from invenio.legacy.bibcirculation.utils
55:1: I101 Imported names are in the wrong order. Should be convert_datestruct_to_dategui, convert_datetext_to_datestruct
56:1: I100 Imports statements are in the wrong order. from invenio.legacy.bibsched.bibtask should be before from invenio.utils.date
59:38: E261 at least two spaces before inline comment
64:1: E302 expected 2 blank lines, found 1
64: D401 First line should be imperative: 'Return', not 'Returns'
64: D400 First line should end with '.', not 'r'
64: D205 Blank line missing between one-line summary and description
88:25: E128 continuation line under-indented for visual indent
92:29: E128 continuation line under-indented for visual indent
92:61: E251 unexpected spaces around keyword / parameter equals
93:29: E128 continuation line under-indented for visual indent
94:29: E128 continuation line under-indented for visual indent
105:27: E128 continuation line under-indented for visual indent
108: D401 First line should be imperative: 'Return', not 'Returns'
108: D202 No blank lines allowed *after* function docstring, found 1
130:29: E128 continuation line under-indented for visual indent
131:29: E128 continuation line under-indented for visual indent
145: D290 White spaces around docstring should be trimmed.
145: D209 Put multi-line docstring closing quotes on separate line
145: D400 First line should end with '.', not 's'
145: D205 Blank line missing between one-line summary and description
154:29: E128 continuation line under-indented for visual indent
155:29: E126 continuation line over-indented for hanging indent
161:27: E128 continuation line under-indented for visual indent
164: D401 First line should be imperative: 'Return', not 'Returns'
164: D202 No blank lines allowed *after* function docstring, found 1
164: D400 First line should end with '.', not 't'
164: D205 Blank line missing between one-line summary and description
183:17: E128 continuation line under-indented for visual indent
184:29: E128 continuation line under-indented for visual indent
187: D401 First line should be imperative: 'Return', not 'Returns'
187: D400 First line should end with '.', not 't'
187: D205 Blank line missing between one-line summary and description
209:13: E128 continuation line under-indented for visual indent
212:13: E128 continuation line under-indented for visual indent
212:44: E251 unexpected spaces around keyword / parameter equals
213:13: E128 continuation line under-indented for visual indent
217: D401 First line should be imperative: 'Return', not 'Returns'
217: D400 First line should end with '.', not 'n'
217: D205 Blank line missing between one-line summary and description
237:21: E128 continuation line under-indented for visual indent
238:21: E128 continuation line under-indented for visual indent
242:21: E128 continuation line under-indented for visual indent
243:21: E128 continuation line under-indented for visual indent
247:25: E128 continuation line under-indented for visual indent
249:25: E128 continuation line under-indented for visual indent
258: D401 First line should be imperative: 'Return', not 'Returns'
258: D400 First line should end with '.', not 't'
258: D205 Blank line missing between one-line summary and description
281:17: E128 continuation line under-indented for visual indent
301:27: E128 continuation line under-indented for visual indent
304: D401 First line should be imperative: 'Return', not 'Returns'
304: D400 First line should end with '.', not 't'
304: D205 Blank line missing between one-line summary and description
322:13: E128 continuation line under-indented for visual indent
325: D401 First line should be imperative: 'Return', not 'Returns'
325: D400 First line should end with '.', not 'r'
325: D205 Blank line missing between one-line summary and description
347:17: E128 continuation line under-indented for visual indent
351:17: E128 continuation line under-indented for visual indent
358: D400 First line should end with '.', not ':'
358: D205 Blank line missing between one-line summary and description
429:37: E126 continuation line over-indented for hanging indent
430:10: E261 at least two spaces before inline comment
467: D400 First line should end with '.', not ':'
467: D205 Blank line missing between one-line summary and description
576:62: E502 the backslash is redundant between brackets
577:23: E128 continuation line under-indented for visual indent
588:13: E128 continuation line under-indented for visual indent
589:13: E128 continuation line under-indented for visual indent
590:13: E128 continuation line under-indented for visual indent
591:13: E128 continuation line under-indented for visual indent
597:13: E128 continuation line under-indented for visual indent
598:13: E128 continuation line under-indented for visual indent
599:13: E128 continuation line under-indented for visual indent
600:13: E128 continuation line under-indented for visual indent
601:13: E128 continuation line under-indented for visual indent
606: D400 First line should end with '.', not ':'
606: D205 Blank line missing between one-line summary and description
674: D400 First line should end with '.', not ':'
674: D205 Blank line missing between one-line summary and description
700: D401 First line should be imperative: 'Return', not 'Returns'
700: D400 First line should end with '.', not 's'
718:17: E128 continuation line under-indented for visual indent
720:17: E128 continuation line under-indented for visual indent
724:17: E128 continuation line under-indented for visual indent
725:17: E128 continuation line under-indented for visual indent
729:25: E128 continuation line under-indented for visual indent
731:25: E128 continuation line under-indented for visual indent
740: D400 First line should end with '.', not ':'
740: D205 Blank line missing between one-line summary and description
838: D400 First line should end with '.', not ':'
838: D205 Blank line missing between one-line summary and description
867:21: E127 continuation line over-indented for visual indent
898: D401 First line should be imperative: 'Return', not 'Returns'
898: D400 First line should end with '.', not 't'
898: D205 Blank line missing between one-line summary and description
944:19: E128 continuation line under-indented for visual indent
949:19: E128 continuation line under-indented for visual indent
949:66: E225 missing whitespace around operator
953:25: E128 continuation line under-indented for visual indent
955:25: E128 continuation line under-indented for visual indent
964: D400 First line should end with '.', not ':'
964: D205 Blank line missing between one-line summary and description
1015: D202 No blank lines allowed *after* function docstring, found 1
1015: D400 First line should end with '.', not ':'
1015: D205 Blank line missing between one-line summary and description
1080: D400 First line should end with '.', not ':'
1080: D205 Blank line missing between one-line summary and description
1123:57: E128 continuation line under-indented for visual indent
1138:9: E128 continuation line under-indented for visual indent
1141: D400 First line should end with '.', not ':'
1141: D205 Blank line missing between one-line summary and description
1200: D400 First line should end with '.', not ':'
1200: D205 Blank line missing between one-line summary and description
1236: D400 First line should end with '.', not ':'
1236: D205 Blank line missing between one-line summary and description
1282:1: E302 expected 2 blank lines, found 1
1343: D400 First line should end with '.', not 'n'
1343: D205 Blank line missing between one-line summary and description
1355:10: E261 at least two spaces before inline comment
1355:11: E262 inline comment should start with '# '
1377:83: E502 the backslash is redundant between brackets
1377:82: E211 whitespace before '('
1378:21: E128 continuation line under-indented for visual indent
1380:82: E502 the backslash is redundant between brackets
1380:81: E211 whitespace before '('
1381:21: E128 continuation line under-indented for visual indent
1383:82: E502 the backslash is redundant between brackets
1383:81: E211 whitespace before '('
1384:21: E128 continuation line under-indented for visual indent
1387:82: E502 the backslash is redundant between brackets
1387:81: E211 whitespace before '('
1388:21: E128 continuation line under-indented for visual indent
1395:1: E302 expected 2 blank lines, found 1
1395: D401 First line should be imperative: 'Build', not 'Builds'
1395: D400 First line should end with '.', not 'r'
1395: D205 Blank line missing between one-line summary and description
1447:1: E302 expected 2 blank lines, found 1
1447: D205 Blank line missing between one-line summary and description
1457:13: E127 continuation line over-indented for visual indent
1460:1: E302 expected 2 blank lines, found 1
1460: D401 First line should be imperative: 'Fetche', not 'Fetches'
1460: D400 First line should end with '.', not 't'
1460: D205 Blank line missing between one-line summary and description
1472:15: E711 comparison to None should be 'if cond is None:'
1472:15: E225 missing whitespace around operator
1472:32: E225 missing whitespace around operator
1476:5: B901 blind except: statement
1494:1: E302 expected 2 blank lines, found 1
1494: D401 First line should be imperative: 'Return', not 'Returns'
1494: D205 Blank line missing between one-line summary and description
1511:73: E502 the backslash is redundant between brackets
1512:25: E127 continuation line over-indented for visual indent
1516:77: E502 the backslash is redundant between brackets
1517:25: E127 continuation line over-indented for visual indent
1527:1: E302 expected 2 blank lines, found 1
1527: D401 First line should be imperative: 'Return', not 'Returns'
1527: D400 First line should end with '.', not 'n'
1527: D205 Blank line missing between one-line summary and description
1561:33: E128 continuation line under-indented for visual indent
1562:28: E211 whitespace before '('
1568:33: E128 continuation line under-indented for visual indent
1569:28: E211 whitespace before '('
1577:28: E211 whitespace before '('
1586:33: E128 continuation line under-indented for visual indent
1591:77: E502 the backslash is redundant between brackets
1591:76: E211 whitespace before '('
1592:21: E128 continuation line under-indented for visual indent
1594:1: E302 expected 2 blank lines, found 1
1594: D400 First line should end with '.', not 't'
1594: D205 Blank line missing between one-line summary and description
1607:13: E127 continuation line over-indented for visual indent
1609:1: E302 expected 2 blank lines, found 1
1609:33: E211 whitespace before '('
1609: D401 First line should be imperative: 'Return', not 'Returns'
1609: D205 Blank line missing between one-line summary and description
1618:73: E502 the backslash is redundant between brackets
1618:72: E211 whitespace before '('
1619:21: E128 continuation line under-indented for visual indent
1622:1: E302 expected 2 blank lines, found 1
1622: D401 First line should be imperative: 'Return', not 'Returns'
1622: D400 First line should end with '.', not 'r'
1622: D205 Blank line missing between one-line summary and description
1636:1: E303 too many blank lines (3)
1638:1: E302 expected 2 blank lines, found 3
1638: D401 First line should be imperative: 'Create', not 'Creates'
1638: D400 First line should end with '.', not 's'
1643: D401 First line should be imperative: 'Return', not 'Returns'
1643: D290 White spaces around docstring should be trimmed.
1643: D400 First line should end with '.', not 'g'
1648: D401 First line should be imperative: 'Return', not 'Returns'
1648: D400 First line should end with '.', not ')'
1653: D401 First line should be imperative: 'Return', not 'Returns'
1659: D401 First line should be imperative: 'Return', not 'Returns'
1659: D290 White spaces around docstring should be trimmed.
1659: D400 First line should end with '.', not 'g'
1665:1: E302 expected 2 blank lines, found 1
1665: D401 First line should be imperative: 'Return', not 'Returns'
1665: D400 First line should end with '.', not 'n'
1665: D205 Blank line missing between one-line summary and description
1698:12: E713 test for membership should be 'not in'
1705:34: E127 continuation line over-indented for visual indent
1708:34: E127 continuation line over-indented for visual indent
1711:34: E127 continuation line over-indented for visual indent
1724: D400 First line should end with '.', not 'y'
1724: D205 Blank line missing between one-line summary and description
1757:26: E127 continuation line over-indented for visual indent
1762:114: E261 at least two spaces before inline comment
1766:16: E713 test for membership should be 'not in'
1770:49: E502 the backslash is redundant between brackets
1771:42: E127 continuation line over-indented for visual indent
1773:48: E502 the backslash is redundant between brackets
1774:42: E127 continuation line over-indented for visual indent
1776:53: E502 the backslash is redundant between brackets
1777:42: E127 continuation line over-indented for visual indent
1791:21: E126 continuation line over-indented for hanging indent
1810: D400 First line should end with '.', not 'g'
1810: D205 Blank line missing between one-line summary and description
1824: D400 First line should end with '.', not 'g'
1824: D205 Blank line missing between one-line summary and description
1841:1: E302 expected 2 blank lines, found 1
1841: D401 First line should be imperative: 'Return', not 'Returns'
1841: D202 No blank lines allowed *after* function docstring, found 1
1841: D400 First line should end with '.', not 'r'
1884: D401 First line should be imperative: 'Create', not 'Creates'
1884: D400 First line should end with '.', not 'd'
1884: D205 Blank line missing between one-line summary and description
1948:1: E302 expected 2 blank lines, found 1
1948: D401 First line should be imperative: 'Create', not 'Creates'
1976: D401 First line should be imperative: 'Create', not 'Creates'
1976: D400 First line should end with '.', not 't'
1987:21: E128 continuation line under-indented for visual indent
1995:21: E126 continuation line over-indented for hanging indent
2034:37: E128 continuation line under-indented for visual indent
2043: D401 First line should be imperative: 'Create', not 'Creates'
2043: D400 First line should end with '.', not 'y'
2091:35: E127 continuation line over-indented for visual indent
2115: D401 First line should be imperative: 'Create', not 'Creates'
2115: D400 First line should end with '.', not 'y'
2141:25: E128 continuation line under-indented for visual indent
2166:21: E128 continuation line under-indented for visual indent
2166:60: E225 missing whitespace around operator
2167:21: E126 continuation line over-indented for hanging indent
2205:18: E222 multiple spaces after operator
2223:9: E113 unexpected indentation
2241:9: E113 unexpected indentation
2305:62: E203 whitespace before ':'
2309: D401 First line should be imperative: 'Return', not 'Returns'
2309: D290 White spaces around docstring should be trimmed.
2309: D400 First line should end with '.', not 'a'
2324: D401 First line should be imperative: 'Create', not 'Creates'
2361: D401 First line should be imperative: 'Create', not 'Creates'
2384:46: E502 the backslash is redundant between brackets
2385:21: E128 continuation line under-indented for visual indent
2411:1: E302 expected 2 blank lines, found 1
2411: D401 First line should be imperative: 'Return', not 'Returns'
2411: D400 First line should end with '.', not 'y'
2417: D401 First line should be imperative: 'Return', not 'Returns'
2417: D400 First line should end with '.', not 'y'
2429: D401 First line should be imperative: 'Return', not 'Returns'
2429: D400 First line should end with '.', not 'y'
2437:14: E128 continuation line under-indented for visual indent
2442: D401 First line should be imperative: 'Return', not 'Returns'
2442: D400 First line should end with '.', not 'y'
2447: D401 First line should be imperative: 'Return', not 'Returns'
2447: D400 First line should end with '.', not 'y'
2452: D401 First line should be imperative: 'Return', not 'Returns'
2452: D400 First line should end with '.', not 'y'
2457: D401 First line should be imperative: 'Return', not 'Returns'
2457: D400 First line should end with '.', not 'y'
2464: D401 First line should be imperative: 'Return', not 'Returns'
2464: D400 First line should end with '.', not 'y'
2472: D401 First line should be imperative: 'Return', not 'Returns'
2472: D400 First line should end with '.', not 'y'
2480: D401 First line should be imperative: 'Return', not 'Returns'
2480: D400 First line should end with '.', not 'y'
2488: D401 First line should be imperative: 'Return', not 'Returns'
2488: D400 First line should end with '.', not 's'
2492:10: E128 continuation line under-indented for visual indent
2496:1: E302 expected 2 blank lines, found 1
2496: D401 First line should be imperative: 'Export', not 'Exports'
2509: D401 First line should be imperative: 'Export', not 'Exports'
2519:66: E502 the backslash is redundant between brackets
2520:18: E128 continuation line under-indented for visual indent
2524: D401 First line should be imperative: 'Export', not 'Exports'
2544:68: E502 the backslash is redundant between brackets
2554:1: E302 expected 2 blank lines, found 1
2554: D400 First line should end with '.', not 'a'
2554: D205 Blank line missing between one-line summary and description
2563:64: E502 the backslash is redundant between brackets
2564:25: E127 continuation line over-indented for visual indent
2567: D400 First line should end with '.', not ','
2567: D205 Blank line missing between one-line summary and description
2622:13: E128 continuation line under-indented for visual indent
2623:13: E128 continuation line under-indented for visual indent
2624:13: E128 continuation line under-indented for visual indent
2625:13: E128 continuation line under-indented for visual indent
2626:13: E128 continuation line under-indented for visual indent
2627:13: E124 closing bracket does not match visual indentation
2627:13: E125 continuation line with same indent as next logical line
2643:13: E125 continuation line with same indent as next logical line
2649: D401 First line should be imperative: 'Return', not 'Returns'
2675:27: E128 continuation line under-indented for visual indent
2678: D401 First line should be imperative: 'Return', not 'Returns'
2678: D400 First line should end with '.', not ','
2678: D205 Blank line missing between one-line summary and description
2722:1: E302 expected 2 blank lines, found 1
2722: D401 First line should be imperative: 'Transform', not 'Transforms'
2722: D400 First line should end with '.', not 'e'
2722: D200 One-line docstring should not occupy 3 lines
2729: D401 First line should be imperative: 'Run', not 'Runs'
2729: D400 First line should end with '.', not 's'
2729: D205 Blank line missing between one-line summary and description
2743: D401 First line should be imperative: 'Return', not 'Returns'
2743: D400 First line should end with '.', not 'n'
2751: D401 First line should be imperative: 'Return', not 'Returns'
2751: D400 First line should end with '.', not 'm'
2760:13: E127 continuation line over-indented for visual indent
2763: D401 First line should be imperative: 'Return', not 'Returns'
2763: D400 First line should end with '.', not 'm'
2767:9: E125 continuation line with same indent as next logical line
2772: D401 First line should be imperative: 'Return', not 'Returns'
2772: D400 First line should end with '.', not 't'
2780: D401 First line should be imperative: 'Return', not 'Returns'
2780: D400 First line should end with '.', not 's'
2784:33: E713 test for membership should be 'not in'
2784:54: E261 at least two spaces before inline comment
2789: D401 First line should be imperative: 'Return', not 'Returns'
2789: D400 First line should end with '.', not 'm'
2797: D400 First line should end with '.', not 'e'
2797: D200 One-line docstring should not occupy 3 lines
2809:1: E302 expected 2 blank lines, found 1
2815:1: E302 expected 2 blank lines, found 1
2820:1: E302 expected 2 blank lines, found 1
2827:1: E302 expected 2 blank lines, found 1
2833:1: E302 expected 2 blank lines, found 1
2846:1: E302 expected 2 blank lines, found 1
2851:37: E128 continuation line under-indented for visual indent
2853:10: E128 continuation line under-indented for visual indent
2854:10: E128 continuation line under-indented for visual indent
2855:10: E128 continuation line under-indented for visual indent
2856:10: E128 continuation line under-indented for visual indent
2857:10: E128 continuation line under-indented for visual indent
2858:10: E128 continuation line under-indented for visual indent


1: D100 Docstring missing
2: L102 copyright year is outdated, expected 2015 but got 2014
21:1: I101 Imported names are in the wrong order. Should be cgi, datetime, os, urllib
21:16: E401 multiple imports on one line
23:1: I101 Imported names are in the wrong order. Should be CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_URL, CFG_WEBDIR
23:1: I201 Missing newline before sections or imports.
24:6: E126 continuation line over-indented for hanging indent
30:1: E302 expected 2 blank lines, found 1
30: D101 Docstring missing
32: D401 First line should be imperative: 'Generate', not 'Generates'
32: D200 One-line docstring should not occupy 3 lines
41: D401 First line should be imperative: 'Generate', not 'Generates'
41: D400 First line should end with '.', not 'r'
41: D205 Blank line missing between one-line summary and description
58:21: E126 continuation line over-indented for hanging indent
66:21: E126 continuation line over-indented for hanging indent
66:74: E502 the backslash is redundant between brackets
67:21: E128 continuation line under-indented for visual indent
76: D401 First line should be imperative: 'Generate', not 'Generates'
76: D200 One-line docstring should not occupy 3 lines
92:33: E126 continuation line over-indented for hanging indent
92:64: E502 the backslash is redundant between brackets
93:33: E128 continuation line under-indented for visual indent
97:33: E126 continuation line over-indented for hanging indent
97:64: E502 the backslash is redundant between brackets
98:33: E128 continuation line under-indented for visual indent
103:21: E128 continuation line under-indented for visual indent
107:5: E303 too many blank lines (2)
107: D401 First line should be imperative: 'Generate', not 'Generates'
107: D200 One-line docstring should not occupy 3 lines
139: D401 First line should be imperative: 'Generate', not 'Generates'
139: D200 One-line docstring should not occupy 3 lines
151:9: E303 too many blank lines (2)
162: D401 First line should be imperative: 'Generate', not 'Generates'
162: D400 First line should end with '.', not 's'
162: D200 One-line docstring should not occupy 3 lines
169: D400 First line should end with '.', not 'r'
169: D200 One-line docstring should not occupy 3 lines
178: D401 First line should be imperative: 'Generate', not 'Generates'
178: D400 First line should end with '.', not 's'
178: D200 One-line docstring should not occupy 3 lines
202:8: E122 continuation line missing indentation or outdented
203:8: E122 continuation line missing indentation or outdented
209: D200 One-line docstring should not occupy 3 lines
217: D400 First line should end with '.', not 't'
217: D200 One-line docstring should not occupy 3 lines
227: D200 One-line docstring should not occupy 3 lines
239:25: E126 continuation line over-indented for hanging indent
239:57: E502 the backslash is redundant between brackets
240:25: E128 continuation line under-indented for visual indent
243: D400 First line should end with '.', not 't'
243: D205 Blank line missing between one-line summary and description
267:26: E711 comparison to None should be 'if cond is None:'
275:17: E711 comparison to None should be 'if cond is None:'
275:17: E225 missing whitespace around operator
280:19: E271 multiple spaces after keyword
282: D401 First line should be imperative: 'Generate', not 'Generates'
282: D200 One-line docstring should not occupy 3 lines
298:33: E126 continuation line over-indented for hanging indent
298:64: E502 the backslash is redundant between brackets
299:33: E128 continuation line under-indented for visual indent
303:33: E126 continuation line over-indented for hanging indent
303:64: E502 the backslash is redundant between brackets
304:33: E128 continuation line under-indented for visual indent
309: D401 First line should be imperative: 'Generate', not 'Generates'
309: D200 One-line docstring should not occupy 3 lines
320:25: E127 continuation line over-indented for visual indent
323:17: E127 continuation line over-indented for visual indent
326: D102 Docstring missing
336: D200 One-line docstring should not occupy 3 lines
376: D401 First line should be imperative: 'Provide', not 'Provides'
376: D200 One-line docstring should not occupy 3 lines
382: D401 First line should be imperative: 'Generate', not 'Generates'
407:72: E261 at least two spaces before inline comment
408:21: E128 continuation line under-indented for visual indent
408:54: E261 at least two spaces before inline comment
409:27: E261 at least two spaces before inline comment
410:60: E261 at least two spaces before inline comment
411:26: E261 at least two spaces before inline comment
414:72: E261 at least two spaces before inline comment
415:21: E128 continuation line under-indented for visual indent
415:54: E261 at least two spaces before inline comment
416:27: E261 at least two spaces before inline comment
417:36: E261 at least two spaces before inline comment
418:50: E261 at least two spaces before inline comment
421:58: E261 at least two spaces before inline comment
422:21: E128 continuation line under-indented for visual indent
422:36: E261 at least two spaces before inline comment
439: D401 First line should be imperative: 'Generate', not 'Generates'
474:72: E261 at least two spaces before inline comment
475:21: E128 continuation line under-indented for visual indent
475:54: E261 at least two spaces before inline comment
476:27: E261 at least two spaces before inline comment
477:60: E261 at least two spaces before inline comment
478:26: E261 at least two spaces before inline comment
481:72: E261 at least two spaces before inline comment
482:21: E128 continuation line under-indented for visual indent
482:54: E261 at least two spaces before inline comment
483:27: E261 at least two spaces before inline comment
484:36: E261 at least two spaces before inline comment
485:50: E261 at least two spaces before inline comment
490:17: E128 continuation line under-indented for visual indent
508:53: E128 continuation line under-indented for visual indent
509:53: E128 continuation line under-indented for visual indent
510:53: E128 continuation line under-indented for visual indent
511:53: E128 continuation line under-indented for visual indent
514:53: E128 continuation line under-indented for visual indent
515:53: E128 continuation line under-indented for visual indent
516:53: E128 continuation line under-indented for visual indent
517:53: E128 continuation line under-indented for visual indent
522:95: E502 the backslash is redundant between brackets
534:13: E128 continuation line under-indented for visual indent
541:45: E128 continuation line under-indented for visual indent
544:21: E128 continuation line under-indented for visual indent
586:25: E128 continuation line under-indented for visual indent
587:25: E128 continuation line under-indented for visual indent
588:25: E128 continuation line under-indented for visual indent
589:25: E128 continuation line under-indented for visual indent
590:25: E128 continuation line under-indented for visual indent
591:25: E128 continuation line under-indented for visual indent
592:25: E128 continuation line under-indented for visual indent
592:77: E502 the backslash is redundant between brackets
593:33: E127 continuation line over-indented for visual indent
703: D401 First line should be imperative: 'Display', not 'Displays'
703: D400 First line should end with '.', not 'd'
711: D401 First line should be imperative: 'Display', not 'Displays'
711: D400 First line should end with '.', not 'd'
715:37: E128 continuation line under-indented for visual indent
719: D401 First line should be imperative: 'Display', not 'Displays'
719: D400 First line should end with '.', not 'd'
727: D400 First line should end with '.', not 's'
736:53: E128 continuation line under-indented for visual indent
743: D400 First line should end with '.', not ')'
763:9: E303 too many blank lines (2)
804: D401 First line should be imperative: 'Generate', not 'Generates'
804: D400 First line should end with '.', not ')'
804: D205 Blank line missing between one-line summary and description
814: D401 First line should be imperative: 'Aggregate', not 'Aggregates'
814: D400 First line should end with '.', not 'e'
814: D205 Blank line missing between one-line summary and description
882: D401 First line should be imperative: 'Generate', not 'Generates'
882: D202 No blank lines allowed *after* method docstring, found 2
908:9: E303 too many blank lines (2)
936:21: E128 continuation line under-indented for visual indent
937:33: E128 continuation line under-indented for visual indent
980: D401 First line should be imperative: 'Generate', not 'Generates'


1: D400 First line should end with '.', not 'o'
1: D200 One-line docstring should not occupy 3 lines
2: L102 copyright year is outdated, expected 2015 but got 2014
24:1: I100 Imports statements are in the wrong order. import cgi should be before import time
27:1: I100 Imports statements are in the wrong order. import sys should be before import urllib
28:1: I100 Imports statements are in the wrong order. import string should be before import sys
31:1: I201 Missing newline before sections or imports.
32:1: I101 Imported names are in the wrong order. Should be CFG_BASE_URL, CFG_INSPIRE_SITE, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_NAME_INTL, CFG_SITE_RECORD, CFG_SITE_SECURE_URL, CFG_SITE_SUPPORT_EMAIL, CFG_SITE_URL, CFG_VERSION, CFG_WEBLINKBACK_TRACKBACK_ENABLED, CFG_WEBSTYLE_TEMPLATE_SKIN
32:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.ext.template
32:1: F401 'CFG_SITE_SECURE_URL' imported but unused
32:1: F401 'CFG_SITE_RECORD' imported but unused
32:1: F401 'CFG_WEBLINKBACK_TRACKBACK_ENABLED' imported but unused
32:1: F401 'CFG_VERSION' imported but unused
32:1: F401 'CFG_WEBSTYLE_TEMPLATE_SKIN' imported but unused
33:6: E126 continuation line over-indented for hanging indent
46:1: I101 Imported names are in the wrong order. Should be gettext_set_language, is_language_rtl, language_list_long
46:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.config
46:1: F401 'is_language_rtl' imported but unused
47:1: I101 Imported names are in the wrong order. Should be create_html_link, get_canonical_and_alternates_urls, make_canonical_urlargd
47:1: F401 'get_canonical_and_alternates_urls' imported but unused
48:30: E126 continuation line over-indented for hanging indent
49:1: I100 Imports statements are in the wrong order. from invenio.utils.date should be before from invenio.utils.url
49:1: F401 'convert_datestruct_to_dategui' imported but unused
50:31: E126 continuation line over-indented for hanging indent
51:1: I100 Imports statements are in the wrong order. from invenio.modules.formatter should be before from invenio.utils.date
53:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.html
57: D101 Docstring missing
59: D202 No blank lines allowed *after* method docstring, found 1
59: D400 First line should end with '.', not 'y'
59: D208 Docstring is over-indented
111: D401 First line should be imperative: 'Create', not 'Creates'
111: D400 First line should end with '.', not 'e'
111: D208 Docstring is over-indented
200: D401 First line should be imperative: 'Create', not 'Creates'
200: D202 No blank lines allowed *after* method docstring, found 1
200: D400 First line should end with '.', not 'r'
200: D208 Docstring is over-indented
261: D401 First line should be imperative: 'Create', not 'Creates'
261: D400 First line should end with '.', not 'r'
261: D208 Docstring is over-indented
290: D202 No blank lines allowed *after* method docstring, found 1
290: D400 First line should end with '.', not 'e'
290: D205 Blank line missing between one-line summary and description
290: D208 Docstring is over-indented
327:18: E126 continuation line over-indented for hanging indent
333: D401 First line should be imperative: 'Produce', not 'Produces'
333: D202 No blank lines allowed *after* method docstring, found 1
333: D208 Docstring is over-indented
369:13: B901 blind except: statement
369:20: E261 at least two spaces before inline comment
389:21: B901 blind except: statement
391:34: E261 at least two spaces before inline comment
441:17: E126 continuation line over-indented for hanging indent
441:28: E203 whitespace before ':'
444:37: E502 the backslash is redundant between brackets
445:24: E131 continuation line unaligned for hanging indent
447:24: E203 whitespace before ':'
448:24: E203 whitespace before ':'
449:31: E502 the backslash is redundant between brackets
449:28: E203 whitespace before ':'
450:21: E131 continuation line unaligned for hanging indent
451:90: E225 missing whitespace around operator
452:28: E203 whitespace before ':'
453:28: E203 whitespace before ':'
454:28: E203 whitespace before ':'
455:28: E203 whitespace before ':'
456:28: E203 whitespace before ':'
457:28: E203 whitespace before ':'
458:28: E203 whitespace before ':'
459:28: E203 whitespace before ':'
460:29: E203 whitespace before ':'
461:62: E502 the backslash is redundant between brackets
461:28: E203 whitespace before ':'
461:37: E225 missing whitespace around operator
462:34: E131 continuation line unaligned for hanging indent
462:67: E502 the backslash is redundant between brackets
463:34: E131 continuation line unaligned for hanging indent
464:15: E126 continuation line over-indented for hanging indent
467: D401 First line should be imperative: 'Print', not 'Prints'
472:53: E251 unexpected spaces around keyword / parameter equals
472:55: E251 unexpected spaces around keyword / parameter equals
472:78: E251 unexpected spaces around keyword / parameter equals
472:80: E251 unexpected spaces around keyword / parameter equals
532:41: E231 missing whitespace after ':'
533:41: E231 missing whitespace after ':'
534:42: E231 missing whitespace after ':'
537:41: E231 missing whitespace after ':'
538:39: E231 missing whitespace after ':'
539:41: E231 missing whitespace after ':'
540:42: E231 missing whitespace after ':'
549:9: E303 too many blank lines (2)
571:9: E303 too many blank lines (2)
583:30: E231 missing whitespace after ':'
584:38: E231 missing whitespace after ':'
589: D401 First line should be imperative: 'Print', not 'Prints'
619:69: E502 the backslash is redundant between brackets
623:41: E231 missing whitespace after ','
633:21: E203 whitespace before ':'
634:19: E203 whitespace before ':'
635:17: E126 continuation line over-indented for hanging indent
644:5: E303 too many blank lines (2)
644: D401 First line should be imperative: 'Display', not 'Displays'
644: D400 First line should end with '.', not 'd'
644: D205 Blank line missing between one-line summary and description
644: D208 Docstring is over-indented
662:9: F841 local variable '_' is assigned to but never used
685:9: E122 continuation line missing indentation or outdented
686:9: E122 continuation line missing indentation or outdented
686:13: E231 missing whitespace after ':'
687:9: E122 continuation line missing indentation or outdented
687:16: E231 missing whitespace after ':'
688:9: E122 continuation line missing indentation or outdented
689:9: E122 continuation line missing indentation or outdented
689:18: E231 missing whitespace after ':'
690:9: E122 continuation line missing indentation or outdented
705:21: E203 whitespace before ':'
706:22: E203 whitespace before ':'
707:22: E203 whitespace before ':'
708:21: E203 whitespace before ':'
709:93: E203 whitespace before ':'
713: D401 First line should be imperative: 'Produce', not 'Produces'
713: D202 No blank lines allowed *after* method docstring, found 1
713: D400 First line should end with '.', not 't'
725:9: F841 local variable '_' is assigned to but never used
729: D401 First line should be imperative: 'Return', not 'Returns'
729: D202 No blank lines allowed *after* method docstring, found 1


1: D100 Docstring missing
2: L102 copyright year is outdated, expected 2015 but got 2014
23:1: I100 Imports statements are in the wrong order. import errno should be before import os
25:1: I100 Imports statements are in the wrong order. import cgi should be before import time
27:1: I100 Imports statements are in the wrong order. import shutil should be before import sys
30:1: I100 Imports statements are in the wrong order. from collections should be before from urllib
33:1: I101 Imported names are in the wrong order. Should be CFG_ACCESS_CONTROL_LEVEL_SITE, CFG_CERN_SITE, CFG_PREFIX, CFG_SITE_LANG, CFG_SITE_NAME, CFG_SITE_SECURE_URL, CFG_SITE_URL, CFG_WEBSUBMIT_STORAGEDIR
33:1: I100 Imports statements are in the wrong order. from invenio.config should be before from invenio.legacy.bibrecord
34:6: E126 continuation line over-indented for hanging indent
42:1: I101 Imported names are in the wrong order. Should be CrossrefError, get_marcxml_for_doi
43:1: I100 Imports statements are in the wrong order. from invenio.utils should be before from invenio.utils.crossref
44:1: I100 Imports statements are in the wrong order. from invenio.legacy.dbquery should be before from invenio.utils
46:1: I100 Imports statements are in the wrong order. from invenio.modules.access.control should be before from invenio.modules.access.engine
47:1: I100 Imports statements are in the wrong order. from invenio.legacy.webpage should be before from invenio.modules.access.control
48:1: I101 Imported names are in the wrong order. Should be collect_user_info, getUid, isGuestUser, page_not_authorized
49:29: E126 continuation line over-indented for hanging indent
51:1: I101 Imported names are in the wrong order. Should be WebInterfaceDirectory, wash_urlargd
51:1: I100 Imports statements are in the wrong order. from invenio.ext.legacy.handler should be before from invenio.modules.records.access
53:1: I100 Imports statements are in the wrong order. from invenio.base.i18n should be before from invenio.utils.url
54:1: I101 Imported names are in the wrong order. Should be decompose_file, propose_next_docname, stream_file
56:1: I100 Imports statements are in the wrong order. from invenio.ext.logging should be before from invenio.legacy.bibdocfile.api
58:1: I101 Imported names are in the wrong order. Should be InvenioWebSubmitIconCreatorError, create_icon
58:1: I100 Imports statements are in the wrong order. from invenio.legacy.websubmit.icon_creator should be before from invenio.utils.html
59:1: I100 Imports statements are in the wrong order. from invenio.legacy.ckeditor.connector should be before from invenio.legacy.websubmit.icon_creator
62:1: F401 'get_detailed_page_tabs' imported but unused
63:1: I101 Imported names are in the wrong order. Should be CFG_JSON_AVAILABLE, json
64:1: I100 Imports statements are in the wrong order. import invenio.legacy.template should be before from invenio.utils.json
65:1: I100 Imports statements are in the wrong order. from flask should be before import invenio.legacy.template
65:1: I201 Missing newline before sections or imports.
70:1: I101 Imported names are in the wrong order. Should be action, endaction, home, interface, makeCataloguesTable
72:1: E302 expected 2 blank lines, found 1
72: D101 Docstring missing
74:77: E502 the backslash is redundant between brackets
75:79: E502 the backslash is redundant between brackets
78: D400 First line should end with '.', not 'r'
78: D205 Blank line missing between one-line summary and description
105:16: E127 continuation line over-indented for visual indent
127:13: B901 blind except: statement
134:9: B901 blind except: statement
141:61: E225 missing whitespace around operator
192:25: F841 local variable 'n' is assigned to but never used
211:34: E201 whitespace after '{'
211:57: E203 whitespace before ':'
212:57: E203 whitespace before ':'
212:69: E261 at least two spaces before inline comment
213:57: E203 whitespace before ':'
214:57: E203 whitespace before ':'
215:57: E203 whitespace before ':'
216:57: E203 whitespace before ':'
216:67: E261 at least two spaces before inline comment
217:57: E203 whitespace before ':'
251:5: E303 too many blank lines (2)
251: D202 No blank lines allowed *after* method docstring, found 1
251: D205 Blank line missing between one-line summary and description
258: D290 White spaces around docstring should be trimmed.
258: D400 First line should end with '.', not 'm'
281:16: E127 continuation line over-indented for visual indent
303:13: B901 blind except: statement
304:17: F841 local variable 'act' is assigned to but never used
373:63: E127 continuation line over-indented for visual indent
416:41: E128 continuation line under-indented for visual indent
417:41: E128 continuation line under-indented for visual indent
418:41: E128 continuation line under-indented for visual indent
422:25: B901 blind except: statement
456:26: E703 statement ends with a semicolon
535: D205 Blank line missing between one-line summary and description
547:16: E127 continuation line over-indented for visual indent
548:16: E127 continuation line over-indented for visual indent
556:9: E303 too many blank lines (2)
569:9: B901 blind except: statement
586:19: E126 continuation line over-indented for hanging indent
590:35: E231 missing whitespace after ','
593:30: E201 whitespace after '{'
593:53: E203 whitespace before ':'
594:53: E203 whitespace before ':'
595:53: E203 whitespace before ':'
596:53: E203 whitespace before ':'
597:53: E203 whitespace before ':'
598:53: E203 whitespace before ':'
598:63: E261 at least two spaces before inline comment
599:53: E203 whitespace before ':'
612:64: F841 local variable 'e' is assigned to but never used
625: D401 First line should be imperative: 'Thi', not 'This'
625: D290 White spaces around docstring should be trimmed.
625: D400 First line should end with '.', not 'g'
625: D205 Blank line missing between one-line summary and description
633:26: E261 at least two spaces before inline comment
634:32: E261 at least two spaces before inline comment
635:33: E113 unexpected indentation
638:47: E261 at least two spaces before inline comment
652: D401 First line should be imperative: 'Return', not 'Returns'
652: D400 First line should end with '.', not 'e'
652: D205 Blank line missing between one-line summary and description
663:9: F841 local variable 'uid' is assigned to but never used
664:9: F841 local variable 'user_info' is assigned to but never used
669:74: E502 the backslash is redundant between brackets
670:62: E502 the backslash is redundant between brackets
670:39: E221 multiple spaces before operator
681: D200 One-line docstring should not occupy 3 lines
709:44: E203 whitespace before ':'
724:9: F841 local variable 'myQuery' is assigned to but never used
755:36: E126 continuation line over-indented for hanging indent
765:22: E211 whitespace before '('
781: D400 First line should end with '.', not ')'
789:37: W602 deprecated form of raising exception
807:32: E203 whitespace before ':'
815: D102 Docstring missing
828:9: F841 local variable 't' is assigned to but never used
829:15: E221 multiple spaces before operator
840:9: F841 local variable 'nbFields' is assigned to but never used
846:24: E126 continuation line over-indented for hanging indent
846:35: E203 whitespace before ':'
847:31: E203 whitespace before ':'
848:30: E203 whitespace before ':'
849:30: E203 whitespace before ':'
850:23: E126 continuation line over-indented for hanging indent
852:59: E231 missing whitespace after ','
855:47: E231 missing whitespace after ','
856:52: E231 missing whitespace after ','
863:18: E126 continuation line over-indented for hanging indent
863:20: E251 unexpected spaces around keyword / parameter equals
863:22: E251 unexpected spaces around keyword / parameter equals
864:24: E251 unexpected spaces around keyword / parameter equals
864:26: E251 unexpected spaces around keyword / parameter equals
865:16: E126 continuation line over-indented for hanging indent
867: D401 First line should be imperative: 'Return', not 'Returns'
867: D200 One-line docstring should not occupy 3 lines
899: D102 Docstring missing
932:45: E127 continuation line over-indented for visual indent
941:62: E203 whitespace before ':'
941:109: E203 whitespace before ':'
941:129: E203 whitespace before ','
942:48: E126 continuation line over-indented for hanging indent
945:41: E128 continuation line under-indented for visual indent
950:62: E203 whitespace before ':'
950:109: E203 whitespace before ':'
950:129: E203 whitespace before ','
951:48: E126 continuation line over-indented for hanging indent
954:41: E128 continuation line under-indented for visual indent
964:62: E203 whitespace before ':'
964:109: E203 whitespace before ':'
964:129: E203 whitespace before ','
965:45: E126 continuation line over-indented for hanging indent
974:27: E225 missing whitespace around operator


OK


Hello there!)
        Sorry, but i didn't get what did you mean about "content" or
"context", where it was?
       Thank you for the link of  validation. I checked my files and found
reasons of errors and I had
solved them...

2015-01-22 12:12 GMT+02:00 Anton Yatsenko <notifications@github.com>:

> good job man
> use http://validator.w3.org to check your html, analyze errors and
> explain to me what you've done wrong
> keep going!
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/LeoBest/LeoBest/commit/67c87192003835baee0f03f04714f21e488072ae#commitcomment-9393834>
> .
>


Can you please not force push dependency updates? It's a real headache if I want to update my fork... :disappointed:


пробел


@makasim This is actually not a test. It have to be done this way.


Add a space after the quote and the word - front.


Thanks


These two methods you added are good. One way you can make this class better is to take the strings you hardcode here, and replace them with "constants", aka variables that you set at the top of the class. Like this:

```python
class ReportPath:

    OPEN_RATES_DIR = 'Open_rates_clicks'
    PROFILE_COMPLETEDNESS_DIR = 'Profile_completedness'
    UNSUBSCRIBES_DIR = 'Unsubscribes_opt_outs'

    def get_open_rate_directory(self):
        open_rate_directory = os.path.join(self._basepath, OPEN_RATES_DIR)  # replaced with constant
        return open_rate_directory

# other methods below etc...
```


Maybe for mobile we could style the info-float differently. The prefixes for problem limits are too long for a mobile screen, so perhaps we should switch to using some icons to display them.

Time = ![https://www.iconfinder.com/icons/85792/stopwatch_icon#size=30](https://cdn1.iconfinder.com/data/icons/iconbeast-lite/30/stopwatch.png) <https://www.iconfinder.com/icons/85792/stopwatch_icon#size=30>
Memory = ![https://cdn3.iconfinder.com/data/icons/google-material-design-icons/48/ic_memory_48px-32.png](https://cdn3.iconfinder.com/data/icons/google-material-design-icons/48/ic_memory_48px-32.png) <https://www.iconfinder.com/icons/352538/memory_icon#size=32>
Points = ![https://cdn2.iconfinder.com/data/icons/flat-ui-icons-24-px/24/checkmark-24-32.png](https://cdn2.iconfinder.com/data/icons/flat-ui-icons-24-px/24/checkmark-24-32.png) <https://www.iconfinder.com/icons/103184/check_checkmark_ok_yes_icon#size=32>

...or something like that, anyways.


да. наоленила


Cool, got it!


Don't leave a broken method. You can "stub out" your methods in python by using the ```pass``` keyword. Change this to...

```python
def make_report_files_dict(self, path):
    pass
```

...until you know how you want to implement it.


Sure, though you still rebase anyway.

I simply don't want to have consequent commits for the same thing...
On Jan 23, 2015 6:40 PM, "Ben Briggs" <notifications@github.com> wrote:

> Can you please not force push dependency updates? It's a real headache if
> I want to update my fork... [image: :disappointed:]
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/giakki/uncss/commit/2a66a9a716c6f37969b98efa1e12dc5965a9324f#commitcomment-9419172>
> .
>


I rechecked the license.  The last PEM file seems to be under the MPL 2.0 license.  That was updated in commit d5e91399.  Not sure what the value of the warnings for 4 lines of code are.


@ldacosta could you point to the commit where this has been fixed?


For the record, Chrome actually does clamp the distance to [refDistance, maxDistance] in all cases.


в планах хочу зв'язати юзерата місце, залайкати та зберегти в улюблене,


That was quick!


Okay. I just tried to build it, but I couldn't find the required proprietary files, like `lss-0.2d-centos6-64.tar.gz`. I'm not sure what to do about this. I don't want to remove an expression that works and that can be useful for people. At the same time, I'm having a hard time using it for anything myself. :-) Maybe it's okay to remove it? Especially since 2.1.x works fine?


OK


it's here, you can comment on particular line


That really cleans up the code.  Looking great.


https://www.dropbox.com/s/eacbgkwkqriketp/Screenshot%202015-01-23%2018.45.21.png?dl=0 <https://www.dropbox.com/s/eacbgkwkqriketp/Screenshot%202015-01-23%2018.45.21.png?dl=0>

A.

> On Jan 23, 2015, at 6:40 PM, Andrej <notifications@github.com> wrote:
>
> Hello there!)
> Sorry, but i didn't get what did you mean about "content" or
> "context", where it was?
> Thank you for the link of validation. I checked my files and found
> reasons of errors and I had
> solved them...
>
>
>
>
>
>
>
> 2015-01-22 12:12 GMT+02:00 Anton Yatsenko <notifications@github.com>:
>
> > good job man
> > use http://validator.w3.org to check your html, analyze errors and
> > explain to me what you've done wrong
> > keep going!
> >
> > —
> > Reply to this email directly or view it on GitHub
> > <https://github.com/LeoBest/LeoBest/commit/67c87192003835baee0f03f04714f21e488072ae#commitcomment-9393834>
> > .
> >
> —
> Reply to this email directly or view it on GitHub <https://github.com/LeoBest/LeoBest/commit/67c87192003835baee0f03f04714f21e488072ae#commitcomment-9419169>.
>


just added some extra cases


Thanks, fixed it in https://github.com/SpongePowered/SpongeAPI/commit/9edab9a8586a3766f47816903890428c62732e3a


Fancy ```lambda```


Did you benchmark the differences? Should be a lot faster now you're not looping through every plugin.


they are the shit


TeamCity MonoGame :: Develop (Win) [Build 3.3.0.1961](http://teamcity.monogame.net/viewLog.html?buildId=5060&buildTypeId=MonoGame_DevelopWin&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 366, ignored: 6 Build time: 00:17:11


should fix #67 , although strictly speaking, options should be optional


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/25_e1b19364e7e16312e4e24937e1ef211fcf9e2930


I'm getting this merge bubble and I'm not sure how to resolve. Do I have to create a commit now? :confused:

```
  "dependencies": {
<<<<<<< HEAD
    "bluebird": "~2.8.2",
=======
    "bluebird": "~2.7.1",
>>>>>>> 098dab8d7b1ed3cb2a0d3b1392ec479e8ab1da3b
```


Is there a reason you removed these `optional_params`?


Ok, maybe I get it.  *Before*, if you had a bad file command, and stop-on-error is false, you would append the null line to the StringBuilder query.  Was this causing an NPE?


The front names should not be site specific. We can keep this for the time being, but this should be generalized - we can discuss about this next week.


Well within the average number of records that I've seen being updated on my
DBs.
average example: 60 series X 2 Seasons X 20 Episodes = 2400 updates.
  _____

From: Js41637 ftw [mailto:notifications@github.com]
Sent: Friday, 23 January 2015 21:14
To: SchizoDuckie/DuckieTV
Cc: garfield69
Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
(f212ab6)

Well I had like 3000 changes and I doubt that could be from a few shows
possibly updating?

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
81ed1348d46bb#commitcomment-9411818> it on GitHub.
<https://github.com/notifications/beacon/AGnK-PcUNgW8qbfRU8MQYkk8tnJ-9FmHks5
nkfo1gaJpZM4DV8kT.gif>


@pepol I'd change this line to "..needed and keep .."


@ldacosta interesting that you need to check for empty


OK


Consider changing this line to:

```ruby
<%= simple_form_for [@topic, @post, :comment] do |f| %>
# or <%= simple_form_for [@topic, @post, Comment.new] do |f| %>
```

This will allow the `persisted?` check to be removed from [this line](https://github.com/dannyvassallo/bloccit/commit/b1f639edd46b4ed919686051c048290f39ed5d75#diff-ab7dbdfb2fa4302d21356a0e615620e3R1), as we'll no longer be adding a blank comment to `@post.comments / @comments`


Oops, didn't noticed that you already added 'JSON'.
But you will break the test, if you add
`test('{"x":5}', '{ JSON.stringify({x: 5}) }')`
in `tmpl.html` test file.
https://github.com/muut/riotjs/pull/175


Mmm, I think you are right. Have you had any problems with this?


@edgemm-dan When you committed straight to the gh-pages repo the other day it totally killed the ability to style any other updates or push content changes to the site.


There should not be any updates if no data has changed. 2500+ seems odd.
On 23 Jan 2015 17:49, "garfield69" <notifications@github.com> wrote:

> Well within the average number of records that I've seen being updated on
> my
> DBs.
> average example: 60 series X 2 Seasons X 20 Episodes = 2400 updates.
> _____
>
> From: Js41637 ftw [mailto:notifications@github.com]
> Sent: Friday, 23 January 2015 21:14
> To: SchizoDuckie/DuckieTV
> Cc: garfield69
> Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
> (f212ab6)
>
>
>
> Well I had like 3000 changes and I doubt that could be from a few shows
> possibly updating?
>
> -
> Reply to this email directly or view
> <
> https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
> 81ed1348d46bb#commitcomment-9411818> it on GitHub.
> <
> https://github.com/notifications/beacon/AGnK-PcUNgW8qbfRU8MQYkk8tnJ-9FmHks5
> nkfo1gaJpZM4DV8kT.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a6681ed1348d46bb#commitcomment-9419356>
> .
>


Shouldn't that be called either:

*Rebellenmarkt
or
*Rebellen-Markt


:crying_cat_face:
But #87 fixed #79 so it not completely out of :dart:


Yeah, I like the idea. Looks good.


Same as the "Rebellen Markt" thing. According to German grammatical rules, this would be either one word or connected with a hyphen.


Could you change the setting from "chamilo_use_cookie_warning_validation" to "cookie_warning" please? (I think the chamilo and validation parts are not necessary to understand what it is, and we never use "chamilo" in settings names)


That's a good idea, @clebergnu !


Caused regression https://github.com/jenkinsci/docker-plugin/pull/143


I'll do it myself. I just saw you were'nt available at the moment


@ldacosta fyi, note that by the time you call ``run`` the value has already been calculated.
also, why do we not propagate the ``Writer`` values instead of the tuple? i.e., why do we call ``run`` at all? if you expose the ``Writer`` interface it's clear that the first type is that of what is being logged and the second is that of the actual value. reducing it to tuple loses that distinction (since it can be any tuple)


@magnayn Could you please next time do it with PR that other people can review?


Will add methods in next release


fixed #24


OK for timer function. commented separately for Writer monad


I changed names of variables in all test functions


fixed


Nice :)


Mhh... the `for` loop could probably be replaced by `wxAuiDoInsertPage`, what about you?


Nice work here @jacknagel.


應該跟原本寫 `> li`，這樣比較準確，不然以後下面階層多了 li 也會被影響到。


Thanks!


I suspect that Trakt.TV are still repairing their data resulting from the
broken migration during their servers upgrade.
I expect the number to records updated to reduce over time to just genuine
updates.
I'll keep an eye out and if the numbers remain high over during the next
week then perhaps it indicates a review of the favoritesService code is
warranted.

  _____

From: SchizoDuckie [mailto:notifications@github.com]
Sent: Saturday, 24 January 2015 05:52
To: SchizoDuckie/DuckieTV
Cc: garfield69
Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
(f212ab6)

There should not be any updates if no data has changed. 2500+ seems odd.
On 23 Jan 2015 17:49, "garfield69" <notifications@github.com> wrote:

> Well within the average number of records that I've seen being updated on
> my
> DBs.
> average example: 60 series X 2 Seasons X 20 Episodes = 2400 updates.
> _____
>
> From: Js41637 ftw [mailto:notifications@github.com]
> Sent: Friday, 23 January 2015 21:14
> To: SchizoDuckie/DuckieTV
> Cc: garfield69
> Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
> (f212ab6)
>
>
>
> Well I had like 3000 changes and I doubt that could be from a few shows
> possibly updating?
>
> -
> Reply to this email directly or view
> <
>
https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
> 81ed1348d46bb#commitcomment-9411818> it on GitHub.
> <
>
https://github.com/notifications/beacon/AGnK-PcUNgW8qbfRU8MQYkk8tnJ-9FmHks5
> nkfo1gaJpZM4DV8kT.gif>
>
> -
> Reply to this email directly or view it on GitHub
>
<https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
81ed1348d46bb#commitcomment-9419356>
> .
>

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
81ed1348d46bb#commitcomment-9419404> it on GitHub.
<https://github.com/notifications/beacon/AGnK-DLyCk_mGUovFHBiDEUjgOXIg7Tdks5
nknOjgaJpZM4DV8kT.gif>


@rdlowrey @bbroerman30 looks like this change broke quite something on windows. Tests like ext/openssl/tests/bug65538_001.phpt do now hang in the client part in the file_get_contents(). When I pass timeout or set it globally, it does the job. But generally the client doesn't respect that the server part has closed the connection. However when the connection is closed, it should have precedence over any timeout settings, no? Haven't debug it deeper yet, nevertheless - any comments on this?

Thanks.


@ldacosta  see the definition of [``toString``](https://github.com/ypg-data/canpipeimport/commit/3cfc8f91bb9373ac926f1bd08cc8618b7f25d2c8#diff-e22c2c56d300cfd316b83a8eb381bdbaR84) it too uses similar strings. why do we have two copies of such similar strings?


Hostování v kraji s sebou nenese žádná volební práva. Pro volební systém je závazný jen domovský kraj podle trvalého bydliště.


This change seems bit unrelated.  Consider putting it in it's own commit.


其實這樣就不需要 & 了


I do, just clone the repo, get rid of my revert and run the test suit


這些 & 都不需要


這個 & 也不需要


This commit has effectively broken the tabs in my web app. I can no longer switch between tabs. I'm still wondering why ...


OK


looks like you mispelled canola oil!


so I literally wasted about 1 hour yesterday trying to make that resource load when running from IntelliJ.

My issue is that when running from intelliJ there's no way I can find that properties file's InputStream, I've tried lots of combinations of paths without luck, copied the file, and nothing (for about an hour)

The big issue is that I can't test YouTube Downloads if this can't find the properties file, I was thinking of actually moving it somewhere else.


我不小心漏掉了OAO" 有辦法針對單一  commit 作修改嗎？ 還是只能一個一個推回去…？


</d-responsive-columns>


Ok, took me a minute to see what you were trying to do. Basically add a Network Enable link to the links that show after a successful plugin install, so you don't have to go back to the plugins list to network enable.

You need to pass an action as the 2nd parameter to `wp_nonce_url()`. If you look at `process_plugins()` you'll see it uses `check_admin_referrer()`, and tries to verify that you are either enabling or disabling a plugin.

You can copy the param from the calls in `network_plugin_action_links()`. It should be something like this:

`$newLink = '' . __('Network Enable') . '';`


& 都不需要。
那個 @extend 這樣可以用？


There are these journals as well: https://gist.github.com/rmzelle/d6bb971948077a78cdc7 (see also https://github.com/citation-style-language/utilities/pull/14#issuecomment-70603442) that are somewhat orphaned.


mmmm creo esto es un comentario


jajajaa


I believe I commited my R program to github, but I am not sure.
After clicking a few things I see
"commit ca5f5bcba2631beb764c7a084gg91f8be9bc173a"
Is that it?

I didn't use any git tools - just went on the browser...
Any help appreciated


Nice work :+1:

Can I ask if you plan to raise a pull request against coreos-cloudinit upstream?


Specifically this is the parameter `'enable-plugin_' . $plugin_file`


I think the comma at the end of the string should not be there.


If you really want to contribute, you can still do it. Why don't you want to take the few minutes of effort to download the dependencies and start working? It is not so hard to get that running.


@ldacosta still unanswered


Already fixed


OK


That's the intention of this .... but not fully compliant.

Of course it could be possible to pass in NO client, but then we have to care how everything should be:
Proxy settings, KeepAlives ... etc. pp. .. and that's a real mess, cause that should be done by the application which is using the APIs.

To create a default implementation of a http client is very easy:
HttpClients.createDefault()

But if the API user want more, like using system properties, proxies etc. pp, then the API shouldn't take care of that, but the application should do everything to setup a http client which is needed for the application.

Therefore it's just an API ... just my 5 cents ...


Chamando essas fontes, ainda precisa das fontes que estão dentro do projeto?


I never said it was impossible. I said the chances are reduced. The extra amount of effort to go around and download the dependencies, verify the versions, and ensure it still compiles is ludicrous. I could do it all with one command and be done with it.

The main question is still the following: why disable other users?


O que é isso?


@salbito Did I somehow rename my master branch? I can't create a pull request there's nothing to compare this branch with (and in this case a master branch). No bueno.


Yep


Poderia remover o comentário, já que a class é "sidebar" fica fácil de entender...

Também poderia usar html5, o que acha?


+1

Thank you so much!


Precisa disso?


@seamusabshere obv you know that Rails 4.2 is out


Acho que todo esse bloco com <a> poderia ser melhor endentado...


thx! Testing right now ...


Objexx_Tuning (DeadParrot) - x86_64-Linux-Ubuntu-14.04-gcc-4.8-IntegrationCoverage-Debug: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-72.88%25-red.svg) ![Coverage Badge](http://img.shields.io/badge/coverage%20status-0.0%25-red.svg)


`IF NOT EXISTS` only works with Postgresql 9.3 and beyond.


should this list the available options? and/or enforce one of the enum values? (can we even use an enum here instead of String?)


I've deleted the fork and re-forked. I just think it would be easier if the history of the repository is kept intact, that way Git can rebase automatically without me having to create intermediate commits to resolve force pushes. :+1:


OK


should storageLevel be under streaming?


Não usou nenhum include para não precisar repetir códigos? Tem isso no php não tem?


Since the storage level is the spark core configuration, I kept it here.


@cbrightsci @aylakhan This may cause a lot of wasted memory churn, since you'll be recreating the same geometry objects every time a new one is connected.


Acho que aqui não está muito semântico...


I moved everything to the begging of the files


OK, thanks.  That was a late night right before our demo for Miami when Davecgh started changing the matrix.

we are tracking our issues in this repo:  https://github.com/FactomProject/WorkItems/issues


As your not specifying NO_VALUE_COPY to the add_rule the string will be duplicated.

There's actually no reason to strdup there at all. just use rawpkt + skip.

Keep in mind however if your packet is not null terminated bad things will happen for all of this so you should protect against that.


I think we can add validation here.


Works fine :+1:


Goog idea


done


@darcyclarke This branch really needs updates.


Stack trace:
```
/lib/classes/Swift/Transport/AbstractSmtpTransport.php:_doMailTransaction:422
/lib/classes/Swift/Transport/AbstractSmtpTransport.php:_sendTo:445
/lib/classes/Swift/Transport/AbstractSmtpTransport.php:send:176
/lib/classes/Swift/Mailer.php:send:85
```


Instead of referencing it from the array, it might be a good idea to save this DisplayObject as a property on the view (like this.grid) and reference it that way, because if you later need to change the order that you add things and put something before the grid, children[0] is no longer going to be a correct reference.


@addyosmani: I think we can release 0.4.0 :D


thanks fixed.


Thanks, Guru!


Em alguns lugares vc usou apenas brackground, em outros background-color, seria bacana padronizar...


`isdisjoint`, `issubset`, and `isintersecting` are accurate names, but they require you to think in terms of set operations.

Consider renaming them to: `is_not_in`, `is_in`, `some_are_in`

Which will make their usage seem much more natural:
```yaml
- { role: "nyc-trees.graphite", when: "['test'] | is_not_in(group_names)" }
...
when: "['packer'] | is_in(group_names)"
...
when: "['development', 'test'] | some_are_in(group_names)"
```

Also, given that `issubset` is currently only used with a single element on the LHS, it could be replaced by `isintersecting`.


Put all styles in the sheet:
```_css('#' + _api.id + ' .jwdisplay ' + selector, style);```


Yeah, this is exactly what I wanted to see. Thanks.


org.jboss.weld.exceptions.DeploymentException: WELD-001409 Ambiguous dependencies for type [SessionInfo] with qualifiers [@Default] at injection point [[field] @Inject private org.guvnor.common.services.backend.util.CommentedOptionFactoryImpl.sessionInfo].

 Possible dependencies [[Producer Method [SessionInfo] with qualifiers [@Any @Default] declared as [[method] @Produces @RequestScoped @Default public static org.uberfire.server.cdi.UberFireGeneralFactory.getSessionInfo(AuthenticationService)], Managed Bean [class org.kie.provider.impl.SafeSessionInfoProvider] with qualifiers [@Any @Default]]]

https://github.com/uberfire/uberfire/blob/0.5.x/uberfire-server/src/main/java/org/uberfire/server/cdi/UberFireGeneralFactory.java#L33-L35


Poderia adotar para nomes de class e id, tudo em inglês, fica mais simples na hora de ler...


fixed -> in the beginning


OK thanks a lot I will work on it.


It is still very dirty code for the moment, I will work on it over the weekend


Thanks, but there is already a description for `id`: <https://github.com/gorhill/uBlock/blob/master/dist/description/description-id.txt>.

The best if you think it needs changes is to edit on Crowdin, as I have a script to import whatever is on Crowdin and this means this would overwrite changes which are local to github only.


Yupi!


Aqui usou inglês para shadow e clear, mas em outros lugares não, mais uma vez reforço, padronizar o código é fundamental, não interessa se vai usar padrão x ou y, a questão é que seu código seja igual em todos os momentos :)


pq está usando um id aqui?


How is demo performance? Can the code to manage object vector be fixed next week?


Todo Style em apenas um arquivo, pode ser complexo para manutenção, poderia dividir em fonts, forms, etc...

Assim teríamos mais arquivos com pouco conteúdo, facilitando leitura e manutenção...

o que acha?


Ok, that is ok, it just means that I have to have some documentation with the APIs on how to create the HttpClient and pass that in


:+1:  @WEduardo


You just need to skip those; it's not so hard.


@mikemcquaid @bfontaine any comment on my thought? Or I was over thinking? :smirk_cat:


encumbered strings doesn't changed in code. Is it right?


Oops! Thanks, I'll do it.


Nope, I just forgot. One moment.


You may want to make line 230 in import_transactions.php as:
````
$curEntryId += (($type <> 0) ? 1 : $input_id);
````


I am not getting a crash that I wasn't seeing before. Not sure if it is related to building like this or not. Will report back after some more testing.


slow internet connection so can't sync all the repositories but i always tag the original author to maintain peace


same reason dude


This is fine for now, but it might also be nice to wrap `run_setup` so that it always calls `invalidate_caches()` after running.


This spot and others that use this response all fail if `message.content` is undefined.


Nope, it's your latest addition to the episodes entity:
![image](https://cloud.githubusercontent.com/assets/111710/5879194/4b138e16-a32d-11e4-981e-fc4c0a336ac3.png)


eder, seems like you don't have this commit locally as it should actually resolve this problem. Try building locally kid-identity-session-provider module and it should work properly.


Faltou indentar todo o código, percebi também há bastante repetição de código.
http://thephpbasics.com/tutorial-7-%E2%80%93-include/ Se quiser, nesse link tem uns videos que explicam bem como usar includes para evitar a repetição.


I think it's okay. Before this change, this wouldn't have been a problem:
```ruby
module Foo
  def bar
    'bar'
  end
end

_pry_.config.extend(Foo)
_pry_.config.bar = 'foo'
_pry_.config.bar    # => 'bar'
_pry_.config['bar'] # => 'foo'
```
But OpenStruct has the exact same problem.
:+1: on the diff.


I believe there is some missing in this change. It does not compile:
# gopkg.in/alecthomas/kingpin.v1
../gopkg.in/alecthomas/kingpin.v1/global.go:46: CommandLine.ParseWithFileExpansion undefined (type *Application has no field or method ParseWithFileExpansion)
../gopkg.in/alecthomas/kingpin.v1/global.go:46: not enough arguments in call to MustParse


use the print function?


Read it as `(@banner = arg) if arg`, not `@banner = (arg if arg)`, and you'll see that the assignment is never executed.


Conhece estrutura MVC? Já usou?


same


This localization key is in the MC lang files


See rust-lang/rfcs#565 for discussion about this new requirement.  It's not clear if this will stick or be reverted/changed.  For now, I just provided the simplest possible implementation to make the tests pass.


It affects only to-many relationship that doesn't have inverse relationship specified. Regarding the bug, when there is no inverse relationship specified, you will not get reverse relationship specified in the view so either querying or getting objects directly with ID, you cannot see any to-many relationship without inverse relationship.


Does this consider the MPAL clock, which is 60Hz? (cxd4 made this comment before)

Q) I've never really heard of MPAL. What is it?

A) M-PAL (PAL-M) is the Brazilian 60 Hz PAL format image reception system with 525 scan lines. Since it is a type of PAL, the color information is different, at 60 Hz, it has the same scanning format as NTSC.

More info here: http://n64devkit.square7.ch/info/mpal/mpal.htm


This is autogenerated by FML.


Thanks @darcyclarke!


@AndreyAkinshin добавить бы ещё нормальный пример цитирования БСЭ в BibTeX. У тебя нигде нету?


The Lines 20,22 and 24 are commented out because there was a compilation error due to those lines added automatically when merged with the remote repo.


Hi! Why are you using only one icon for all sizes?


[Huzzah!](http://h5bp.github.io/Front-end-Developer-Interview-Questions/) No problemo :smile:


There aren't unused svg files... you must have one icon for each size, and if you needs to modify something, you modify in svg, not in png files.


@dustalov Нет, нету. А что, в 2015-ом часто цитируют?


is this correct?


I believe creating different CMA device for every heap is not useful, so I made it like this. After running some benchmarks the performance is not worse, however the delay when opening different apps might increase (I have not noticed myself).


You could alias "@id" to "id" in the context to avoid this quoting issue.


Unfortunately, you can't alias @context... but that only happens once per file.


So where did I go wrong?
Was it that when I renamed imdb_id to IMDB_ID I should have put that change
into a v11 update of the episodes table?

  _____

From: SchizoDuckie [mailto:notifications@github.com]
Sent: Saturday, 24 January 2015 06:26
To: SchizoDuckie/DuckieTV
Cc: garfield69
Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
(f212ab6)

Nope, it's your latest addition to the episodes entity:

<https://cloud.githubusercontent.com/assets/111710/5879194/4b138e16-a32d-11e
4-981e-fc4c0a336ac3.png> image

-
Reply to this email directly or view
<https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
81ed1348d46bb#commitcomment-9420016> it on GitHub.
<https://github.com/notifications/beacon/AGnK-KIsZNJp9hYVdYNtADytBsgD-E9Hks5
nknu3gaJpZM4DV8kT.gif>


Objexx_Tuning (DeadParrot) - Win64-Windows-7-VisualStudio-12: Tests Failed

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-85.23%25-red.svg)


You could alias '@type' to 'type' to avoid the quoting issue.


Hey @charlie-ablett - some suggestions for you wrt. your YAML-LD stuff. Hope it's helpful :)


yeah..its working fine..i compiled it with 8 64 14 52 2

On Fri, Jan 23, 2015 at 11:07 PM, Satendra Tiwari <notifications@github.com>
wrote:

> is this correct?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/shailendert/Data-Structures/commit/7576a26eb1ddfb0d1e14257a2b154c838fdba29b#commitcomment-9420144>
> .
>


I hope that my blog is operational.


@audreyt ++ Must be more apparent for CTBlue :beer:


Also added the views.py


@mikelolasagasti Could you please also bump the date and version in the plugin xml files?


I don't think we should remove this section because this is still very much so true.  I currently use this in two production applications.


_getOutputDuration_ would be clearer, don't you think ?


Yay!


The `create_user` and `sign_in` methods could be moved to a module in the `spec/support` directory, and then this module could be `include`d in any feature spec that needs to sign in a user.

```ruby
# This file lives at spec/support/sign_in_helper.rb
module SignInHelper
  def create_user(email, password)
    ...
  end

  def sign_in(email, password)
    ...
  end
end
```

The `sign_in` method will then be accessible to any feature spec that has `include SignInHelper` in it, below the `feature` line, e.g.:

```ruby
describe "Foo can bar" do
  include SignInHelper
  ...
end
```

Eventually you might find that `include SignInHelper` is being duplicated across feature specs and you'd like to DRY that up.

If that happens, take a look and review the `RSpec.configure do |config| ... config.include ... end` blocks in some of the other files in `spec/support`. `config.include` is a way RSpec provides to `include` modules in specs - it could be used with `SignInHelper`.


Hi Taylor, some users asked before about add comments to columns, may be useful add to this schema.md document the following possibility:

    $table->string('name')->comment = "Here goes the column comment";


ok


BTW, I went with 0.4.0 in the CHANGELOG, feel free to change it if you go with another version.


right, thanks, will fix.


To have a label text instead of using the Key as label.


@giakki: please add the GitHub release too otherwise we end up with 404 :P

https://github.com/giakki/uncss/blob/0.12.0/README.md#within-node


The 404 https://github.com/giakki/uncss/blob/0.12.0/README.md#within-node will be fixed when @giakki adds the tag :)


<pre>
if defined(__GNUC__) && !defined(__MINGW32__)
</pre>
is needed for mingw cross-compilation.
Please re-add.


This is the only exception to this commit.

Out of all the Project64 RSP interpreter op-codes, VNAND did not use the `Indx` array.
It just inverted the count by ^= 7.

That obviously was still broken with certain elemental specifiers for shuffling the vector, but according to my tests, none of the N64 commercial games have ever executed a VNAND instruction before (only VAND, VOR, VNOR, VXOR, and VNXOR).  As such, this commit also fixes an apparent bug with the VNAND interpreter op-code, although it has never been encountered in any N64 ROMs according to my own RSP tests.


r+


@schlende thanks bro!  I know your comment was to look out for our best interest.  AWS called me withing like 5 mins so i reset my secret.  Learning lessons the hard way :+1:


Markowitz


> As of PHP 5.4

Ah did not knew that, thanks for the link. PR #2334


I will try to create some kind of reference implementation in the api-common package
of a "default YAMJ http client builder", where proxy settings can be passed in and connection timeout and
using system properties and some kind of that.

When calling the "build()" on that DefaultHttpClientBuilder, then a http client will be build,
which then can be passed into the APIs.

Will then be easier configurable with spring and be in the same manor then the 4.3.x HttpClientBuilder ..
but more user friendly then the http client implementation is cause there you must create objects of request config, passed it in the http client builder as default .. and some other stuff.


Hi,

I just tried your final example a couple of times. In Python 2.7 I get the following in the Python console:

```
>>> import pyscrypt
>>> with pyscrypt.ScryptFile('foo.scrypt', 'pass123', 'seasalt', 1024, 1, 1) as f:
...     f.write('hello, world!')
...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "pyscrypt/file.py", line 148, in __init__
    key = hash(password, salt, N, r, p, 64)
  File "pyscrypt/hash.py", line 235, in hash
    if N < 2 or (N & (N - 1)): raise ValueError('Scrypt N must be a power of 2 greater than 1')
TypeError: unsupported operand type(s) for -: 'str' and 'int'
```

In Python 3.4 I get this:

```
>>> import pyscrypt
>>> with pyscrypt.ScryptFile('foo.scrypt', b'pass123', b'seasalt', 1024, 1, 1) as f:
...     f.write(b'hello, world!')...
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/ntoll/src/pyscrypt/pyscrypt/file.py", line 140, in __init__
    raise ValueError('salt must be a byte array')
ValueError: salt must be a byte array
```

...both of which I wasn't quite expecting. ;-)


Looks good!


Thanks to kashif. Sorry didn't cherry-picked.


Strange it should prevent corruption. Maybe the queue-size was chosen to aggressivly.
I'll change the size to 50 entries.


always use {} even if is a single line?


Bu in the mean time you could change your APIs, so that the also just use the HttpCLient interface
and no default constructor without http client ... therefore I made the implementation in allocine
which still works with YAMJ v3 cause you just pass in a http client interface .. regardless if it's an old
(which is deprecated) or a new one (using 4.3.x http client stuff).

Have a nice weekend ... :-)


ah, makes sense. not sure why i did it async. i guess i'm so used to trying to do everything async these days.


Furthermore, if you have this code somewhere for one reason or another:
```puppet
package { 'sinatra':
  ensure => absent,
}
```
It won't install either.


Redundant whitespace.


i don't understand how this works?  if you don't call done() won't the test timeout?


With regexp implemented now, it's a good way to put this one and the sharesix.com "checkmate" ;)


TeamCity Bau :: Dude [Build 1](http://teamcity.codebetter.com/viewLog.html?buildId=179452&buildTypeId=Bau_Dude&guest=1) is now running


TeamCity Bau :: Dude [Build 1](http://teamcity.codebetter.com/viewLog.html?buildId=179452&buildTypeId=Bau_Dude&guest=1) outcome was **FAILURE**
Summary: Tests failed: 1 (1 new), passed: 0 Build time: 00:00:35
### Failed tests
```
DudeTests.Acceptance.Console.dll: DudeTests.Acceptance.Console.ScriptExecution.HelloWorld: <no details avaliable>

```


Thank you for considering field order. Consider using the same order as our template:

Point X	Point Y	Street Address	City	Postal Code	Plot Width	Plot Length	OpenTreeMap Plot ID	External ID Number	Tree Present	Genus	Species	Cultivar	Other Part of Name	Common Name	Diameter	Tree Height	Canopy Height	Date Planted


Not setting default value here would help configuring the value from external properties (via servers.yml).


But now who does the null termination?


So @addyosmani will you be able to make the release tonight?


Fixed in More Blocks upstream.

<https://gitorious.org/calinou/moreblocks/commit/f0a455f878c8d72865c91799a793d21bb82e9860>


TeamCity Bau :: Dude [Build 2](http://teamcity.codebetter.com/viewLog.html?buildId=179455&buildTypeId=Bau_Dude&guest=1) is now running


TeamCity Bau :: Dude [Build 2](http://teamcity.codebetter.com/viewLog.html?buildId=179455&buildTypeId=Bau_Dude&guest=1) outcome was **SUCCESS**
Summary: Tests passed: 1 Build time: 00:00:14


`log.Fatalln(err)` instead?


Consider using the same field order as our template (or updating the template):

Genus	Species	Cultivar	Other Part of Name	i-Tree Code	Common Name	Is Native	Flowering Period	Fruit or Nut Period	Fall Conspicuous	Flower Conspicuous	Palatable Human	Has Wildlife Value	Fact Sheet URL	Plant Guide URL	Max Diameter	Max Height


I think it is much clearer as you say! I did that because in one of the videos was done in a line


Good call, will change


ChIPpip


This should be indented.


Test.


@tombye

LGTM - only issue is this PR still has the `grunt-contrib-sass`  in the `node_modules` folder. But I guess that's fixed by the other PR where we remove all node modules?


Fix the nesting here.


84202576-DesSpecOA-PerPersonUsedWhenFlowArea (JasonGlazer) - x86_64-MacOS-10.9-clang: OK (1583 of 1585 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.87%25-yellow.svg)


They were.. I was just on the wrong branch. Derp.


The sections aren't necessary around all of the ```p``` tags


nice


You're happy with the state of master at the moment? (if so, yes!)


hey  @keyur2maru
take a look at gtalk or hangouts


Yes, I tested it with my mpc-hc.org repo and worked fine with 0.12.0, hence the update :)


Always add an additional blank line at the end of the document.


If you rename your stubs to add .php extension you can have code highlighting in your editor and on GitHub.


Yeah, I was unsure on that one. But your right, after all, it only has to be typed inside this function, and only in a few places. And as a keyword argument, but that's not a lot of typing, either. I'll probably change that.


Your content should include your article.


well, i see you like option 3


@fredwu Will do! I got permissions to just recently, so I'll push it this evening.


This is probably more of an aside.


What cause the difference is you have a very high-res monitor. Therefore `shouldPushLeft` is false. If it's true and `padding-left` is modified in any page other than split-diff, problem occurs. Try reduce resolution or try a smaller monitor :).


Why is this a menu?


It might make sense now that I think about it to have slugs out of the tags package. I think you'd always use /tags/$id/$slug where slug is like how you did just strtolower and pretty url (narrow the scope) and if people want it they should extend the package.


Perfect. Thanks! Henrik is so sloppy. ;-)


This is over indented.


Thanks!


No tags?


fix jellekralt/Responsive-Tabs#44


Yeah, I wondered that too, as my comments to #1084 show. I verified that few of vstr_* functions take care of that. But now appears vstr_extend() doesn't. So, it's confusing. The best place to put it is in mp_obj_new_str_from_vstr(). I wanted to do just that, or at least put assert, but gave up with just adding another test.


Would it make sense to just put this logic in the hasTag() and check if it is an integer?


Remove all the extra white space in between your elements. Add a new blank line under line 75


84202576-DesSpecOA-PerPersonUsedWhenFlowArea (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-gcc-4.8: OK (1589 of 1591 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg) ![Test Badge](http://img.shields.io/badge/tests%20passed-99.87%25-yellow.svg)


Why a cheat?


My release script had a hiccup. Ignore.


Disregard the second paragraph in my last comment - they must have fixed this issue.

@Calinou You didn't fix the default overrides and default:glass doesn't even need to be overridden.


Fixed. Issue was not the namespace but the class name was wrong. https://github.com/madewithlove/elasticsearcher/commit/58f3d6162c6d6fef61abd10a716b21013f076eaa

// cc @Anahkiasen


are these -ve nos?


the space isn't required, it's actually `/\s*/`


Consider adding TODOs in exporter views which could use this decorator, so we maybe change them some day and so someone working in the code would be aware that it is also used by the importer.


Why has this comment line been removed? It was there in the previous version of this patch and leaves the code with a hanging sentence: "a standby state".


Unlike `it()`, `describe()` does not actually allow for asynchronous execution using its first parameter. The `done` in this case is unused and has no side-effects. I removed it to avoid the confusion here: 72a47b261df77f426c7c86c8636992032eb0a7ae


fix jellekralt/Responsive-Tabs#46


The preview in Instagram camera is just black.


Given that a `Point` can be returned from methods such as  `TextEditor::screenPositionForBufferPosition` (https://atom.io/docs/api/v0.175.0/TextEditor#instance-screenPositionForBufferPosition), naming these fields `row` and `column` does not seem appropriate.


@jonalmeida - can you remember why you made this change? Was it for Android jar size?


Yeah, i tried that first. Since `network_plugin_action_links` calls for `$context`, that was messing it up, so I removed that and hardcoded `&plugin_status=all` but when I leave the `'enable-plugin_' . $plugin_file` line in, I still get a busted URL.


This comes as a surprise! Will renderDepth be featured in any other way or is there an explanation why it has been taken out? Our projects rely on it heavily because objects come in dynamically in no particular order.


This commit is extremely unproductive. If your goal is to discourage developers from getting involved, you're doing a goddamn good job at it. I don't care about factions, but I can tell you that removing maven does NOT help me become more interested one bit.

There is an alternative called Gradle, in my experience it compiles much much faster than Maven. You're honestly better off using Gradle if you're having a time-compilation issue. Please rethink this commit, think of all the devs that won't get involved because of how they need to download dependencies from everywhere and add them to the project.

Also, **please** consider looking into an IDE called [IntelliJ IDEA](https://www.jetbrains.com/idea/). I honestly don't care what IDE you use, but I'm just letting you know that when I switched to IntelliJ I became a ton more productive due to all the tools and fancy stuff it provides a developer.


@mattbostock I kinda do indeed. But there are a few things that need sorting out first:
- I'd like to have the systemd units for CoreOS ready at the same time
- I need to figure out whether there will be licensing issue (open-vm-tools is LGPL, which might be a problem for this otherwise-Apache project)


84202576-DesSpecOA-PerPersonUsedWhenFlowArea (JasonGlazer) - x86_64-Linux-Ubuntu-14.04-cppcheck-1.61: OK (0 of 0 tests passed)

![Build Badge](http://img.shields.io/badge/build%20status-passing-green.svg)


You're right, just couldn't figure out how to make it work! Thanks :+1:


@mattallan I'd separate these into a separate commit to make it clear what you're doing here. At first it looked like you were changing the return value and preventing an exception being thrown, but upon inspection of `WC_API_Resource::validate_request()`, I realised you're just removing duplicate code.

To change it up, you can use `git rebase`. We can go over it together next week if you're not quite sure how to do that. :)


A nav tag is intended to be a parent to all it's links. Use one ```nav``` to wrap your links

```
<nav>
   <a href="#">My link</a>
   <a href="#">My link</a>
</nav>
```


I'm skeptical this will work, unless you put /opt/curl/lib in the shared library path via ldconfig.

I guess we need to embed /opt/curl into the runtime path for the linker.


Calinou's fixes have been brought into Dreambuilder also.


:cake:!


As we discussed, consider making this a function rather than a decorator, so the calling code will be easier to understand.


Marco and I were discussing issues with the single (new module) SessionInfo provider (taking place of UberfireGeneralFactory) as the new module's implementation doesn't support marshaling SessionInfo from server to client (unlike UberfireGeneralFactory's implementation and associated Errai cust marshaller). We're waiting for porcelli to return to discuss further.


Always add a blank line to the end of your documents.


AG will commit


we know it's a list of strings, is this necessary?


technically, this should say 'bytes' instead of 'str'


I created the class using TDD (constructor and getDollarsAmount method), and then I found parts of the code on internet and the rest was implemented by me. I forget to pass convert method to private, but the method getDollarsAmount use all private methods, so are tested.


tweaked settings to prevent Django 1.5 detection in Django 1.7
                   ^
SyntaxError: invalid syntax


AHH, GOT IT. I was using a dot, not a comma. grr.


Well vstr_extend does not conform to standard vstr behaviour, in that it doesn't consider the null byte (while all other functions do).  In fact, it should return buf + alloc - 1, since if you used all available room in the vstr it means you used alloc-1 bytes (1 byte for the null).

> The best place to put it is in mp_obj_new_str_from_vstr().

That's overkill because vstrs are always null terminated, or at least that was how they were designed :)

So, either vstr_extend is made to conform by keeping room for a null, or all of vstr's behaviour changes so that the user has to explicitly add a null byte.  The latter may lead to smaller code if done correctly (since in most places we only need to add the null when converting to a str obj), but there are cases like in the lexer and readline where vstr is assumed to be null terminated at all times.


Como dica, usaria CDN para chamar o js e css de dependências. Tipo bootstrap, jquery, etc.


Sorry, I'm a little confused reading through this, but I think we can make it a little clearer and easier to read. The point of this is to loop through the buttons and update them to the right items. Can we do something like:

let panels = newSetOfPanels
let selectedPanel = buttons[selectedIndex].item

// Update any existing buttons we've got
for (index, button) in enumerate(buttons) {
    if index < panels.count {
        // Make this button show the right panel
        let panel = panels[index]
        button.item = panel // This should be a no-op if its the same as the buttons previous panel
        if panel == selectedPanel { // You'll have to write an equality operator for this to work
             selectedIndex = index
        }
    } else {
        // Remove the button
    }
}

// Now add any new buttons we need
for index in buttons.count..<panels.count {
    let button = new Button(panels[index])
    if panels[index] == selectedPanel {
         selectedIndex  = index;
    }
}

This is pretty similar to the old code we had, but is easier to read than I this I think. Is there a reason it won't work?


I am not sure WTF this was. I simply commented it out. It makes no sense at all.


I think this'd better go into gemspec.


The 'images' object is not getting serialized/deserialized properly. Don't
do anything, i'll patch CRUD.js so that it supports this (which i was
actually planning anyway

No sweat :)

On Fri, Jan 23, 2015 at 6:37 PM, garfield69 <notifications@github.com>
wrote:

> So where did I go wrong?
> Was it that when I renamed imdb_id to IMDB_ID I should have put that change
> into a v11 update of the episodes table?
>
>
> _____
>
> From: SchizoDuckie [mailto:notifications@github.com]
> Sent: Saturday, 24 January 2015 06:26
> To: SchizoDuckie/DuckieTV
> Cc: garfield69
> Subject: Re: [DuckieTV] changes for trakt.tv v2 API Series data #248
> (f212ab6)
>
>
>
> Nope, it's your latest addition to the episodes entity:
>
> <
> https://cloud.githubusercontent.com/assets/111710/5879194/4b138e16-a32d-11e
> 4-981e-fc4c0a336ac3.png> image
>
> -
> Reply to this email directly or view
> <
> https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a66
> 81ed1348d46bb#commitcomment-9420016> it on GitHub.
> <
> https://github.com/notifications/beacon/AGnK-KIsZNJp9hYVdYNtADytBsgD-E9Hks5
> nknu3gaJpZM4DV8kT.gif>
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/SchizoDuckie/DuckieTV/commit/f212ab6a658a6edf0f9a4230a6681ed1348d46bb#commitcomment-9420153>
> .
>


I used this class since we had Base64 encoding done within the java-core and that class wasn't available in Android. I decided to use one single Base64 encoder across both so that it was consistent across both trackers.


Ok, once I got the salt dupe stuff working properly, the format now started to find the WRONG password for a cracked hash, and or miss them. This was found with problems in the TS, and took some single stepping to find the problem.

It appears that when salt hash is used, the cmp_all is never called.  The hash does the cmp_all, SO that means that cmp_one MUST not simply return a 1.   We should look at other formats, to make sure they do not have this problem.  The TS may or may not find the issue (I would hope that it would find it, but now 100% sure).  The symptom was the TS found 1500 records, BUT the -show was shorter than 1500, so I am hoping this is the only case that is a problem.


Rather than using a filter, we should use an object oriented tactic to allow `WC_API_Subscriptions` to set the method for creating the subscription/order.

We could either:
* add a new (`protected`) method to `WC_API_Orders` which is a wrapper for actually calling `wc_create_order()` - this would end up with a bit of a hacky name as the class already has a `create_order()` method so it would be named something like `call_create_order()` or `wc_create_order()`. The advantage here is that child classes have complete control over what happens in that method.
* add a new property in `WC_API_Orders` with the name of the function used to create the order (e.g. `protected $create_order_function = 'wc_create_order'`). The advantage here is that we don't need to use a hacky method name, but the downside is that we use a hacky variable function name (generally not a great idea) and that a function outside of the API needs to take care of all the order creation (fine for our use case, but maybe an issue for others).

The former is probably the better if we can find a suitable name.


Body type is not a thing.


Actually, this single line is the key of this whole patch, and it might be worth always doing loose matching on the release part of the version string and get rid of all the other changes. Let me know if you agree and I will revise the pull request.


ah, sorry I messed things up. What is the criteria class from root namespace? what is this?


Thanks, understood! Now the trackers are fully separate, @jbeemster has been able to switch to using the Android built-in: https://github.com/snowplow/snowplow-android-tracker/commit/a586dafe8a612da3c01229207d66498b6d685080


Review the use of the ```h1``` tag. These are links and not headers I'm assuming.


in the normal case this function return a unicode string but if the first decode happens to fail then it will return a byte string !! :|

we should either return always unicode string or return always byte string !


Please check if 8f9ecba29968b4b89b03b0fcd747325a51db90cc fixes the problem.


@mswiderski I don't think so. It's a Ambiguous dependencies because SafeSessionInfoProvider and UberfireGeneralFactory's produces the same bean.

@manstis OK. For now, if anyone get this exception on kie-wb, just need to comment UberfireGeneralFactory.


Is this is a real use case? Tags would be presented to the user and I can't imagine wanting to see dates and such. Wondering if this should be outside the scope of a tag package.


Links to the PR after a change log item are helpful


Published to http://vui-docs-internal.s3-website-us-east-1.amazonaws.com/26_e492ebcb0cd8a1cdb8b179e19a83ead2e7012425


Make the actual text indent.


And also thanks for adding a change log item. :)


Explain all these sections.