)]}'
{"monascaclient/v2_0/shell.py":[{"author":{"_account_id":16168,"name":"Tomasz Trębski","email":"kornicameister@gmail.com","username":"trebskit"},"change_message_id":"245d078b539f94604d099e017636185a2680f99d","unresolved":false,"context_lines":[{"line_number":485,"context_line":"                formatters\u003dformatters)"},{"line_number":486,"context_line":""},{"line_number":487,"context_line":""},{"line_number":488,"context_line":"def _validate_notification_period(period, notification_type):"},{"line_number":489,"context_line":"    if notification_type !\u003d \u0027WEBHOOK\u0027 and period !\u003d 0:"},{"line_number":490,"context_line":"        print(\"Invalid period, can only be non zero for webhooks\")"},{"line_number":491,"context_line":"        return False"}],"source_content_type":"text/x-python","patch_set":7,"id":"7aa08908_01d7f123","line":488,"range":{"start_line":488,"start_character":4,"end_line":488,"end_character":33},"updated":"2016-06-06 20:15:33.000000000","message":"Just out of curiosity, why is it better here to return false instead of throwing Exception ?","commit_id":"c753513f5b8ea281d8256ab96b822347c67a50ad"}]}