)]}'
{"tools/validate_tags.py":[{"author":{"_account_id":6159,"name":"Flavio Percoco Premoli","display_name":"flaper87","email":"flavio.percoco@flyrlabs.com","username":"flaper87"},"change_message_id":"15bce23a9ec4661e6fd4378b8f0a3d5d02d993e5","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def main():"},{"line_number":47,"context_line":"    filename \u003d os.path.abspath(\u0027../reference/projects.yaml\u0027)"},{"line_number":48,"context_line":"    with open(filename, \u0027r\u0027) as f:"},{"line_number":49,"context_line":"        teams \u003d yaml.load(f.read())"},{"line_number":50,"context_line":"    for team in teams:"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_e503fd0d","line":47,"updated":"2016-03-29 12:31:10.000000000","message":"This really depends on where you\u0027re running the script from. Could you use something different here? For example, get the path of this script and use it to build the path to `projects.yaml`\n\n    # NOT TESTED\n    this_script \u003d os.path.abspath(os.path.dirname(__file__))\n    filename \u003d os.path.abspath(os.path.join(this_script, \u0027../reference/projects.yaml\u0027))","commit_id":"d0ab412f1062d25b932e20109a064f9daa107766"},{"author":{"_account_id":12651,"name":"Shamail Tahir","email":"ItzShamail@gmail.com","username":"shamail"},"change_message_id":"f1b70b3739ee143ac7bc7ffa1d82033b7316a2f2","unresolved":false,"context_lines":[{"line_number":44,"context_line":""},{"line_number":45,"context_line":""},{"line_number":46,"context_line":"def main():"},{"line_number":47,"context_line":"    filename \u003d os.path.abspath(\u0027../reference/projects.yaml\u0027)"},{"line_number":48,"context_line":"    with open(filename, \u0027r\u0027) as f:"},{"line_number":49,"context_line":"        teams \u003d yaml.load(f.read())"},{"line_number":50,"context_line":"    for team in teams:"}],"source_content_type":"text/x-python","patch_set":3,"id":"ba0121b8_f7f8c17e","line":47,"in_reply_to":"ba0121b8_e503fd0d","updated":"2016-03-29 16:18:42.000000000","message":"Thanks Flavio, this approach makes sense.  I\u0027ll test it out and also add a return code for when the file is not found.","commit_id":"d0ab412f1062d25b932e20109a064f9daa107766"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"904dad1611f13d5a8ffd5e1e96481b7ec6076c59","unresolved":false,"context_lines":[{"line_number":46,"context_line":"def main():"},{"line_number":47,"context_line":"    script_path \u003d os.path.abspath(os.path.dirname(__file__))"},{"line_number":48,"context_line":"    filename \u003d os.path.abspath(os.path.join(script_path, \u0027../reference/projects.yaml\u0027))"},{"line_number":49,"context_line":"    if os.path.isfile(filename) \u003d\u003d False:"},{"line_number":50,"context_line":"\tsys.exit(\"Projects.yaml was not found at %s\" % (filename))"},{"line_number":51,"context_line":"    with open(filename, \u0027r\u0027) as f:"},{"line_number":52,"context_line":"        teams \u003d yaml.load(f.read())"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba0121b8_621fae5f","line":49,"updated":"2016-03-30 08:58:12.000000000","message":"nitpick: \"if not os.path.isfile(filename):\"","commit_id":"bb6beb6359dfa7e8ff9719ebccfc2863ce764210"},{"author":{"_account_id":308,"name":"Thierry Carrez","email":"thierry@openstack.org","username":"ttx"},"change_message_id":"904dad1611f13d5a8ffd5e1e96481b7ec6076c59","unresolved":false,"context_lines":[{"line_number":47,"context_line":"    script_path \u003d os.path.abspath(os.path.dirname(__file__))"},{"line_number":48,"context_line":"    filename \u003d os.path.abspath(os.path.join(script_path, \u0027../reference/projects.yaml\u0027))"},{"line_number":49,"context_line":"    if os.path.isfile(filename) \u003d\u003d False:"},{"line_number":50,"context_line":"\tsys.exit(\"Projects.yaml was not found at %s\" % (filename))"},{"line_number":51,"context_line":"    with open(filename, \u0027r\u0027) as f:"},{"line_number":52,"context_line":"        teams \u003d yaml.load(f.read())"},{"line_number":53,"context_line":"    for team in teams:"}],"source_content_type":"text/x-python","patch_set":4,"id":"ba0121b8_df3c93e4","line":50,"updated":"2016-03-30 08:58:12.000000000","message":"Using tab instead of spaces (damn, no pep8 check here)","commit_id":"bb6beb6359dfa7e8ff9719ebccfc2863ce764210"}]}