)]}'
{"keystone/service.py":[{"author":{"_account_id":4,"name":"Dolph Mathews","email":"dolph.mathews@gmail.com","username":"dolph"},"change_message_id":"60b6438809b3a000555379efc91bfefb21fe0a14","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    # Add in the v3 version api"},{"line_number":104,"context_line":"    v3routers.append(routers.VersionV3(\u0027admin\u0027, v3_resources_sources))"},{"line_number":105,"context_line":"    v3routers.append(routers.VersionV3(\u0027public\u0027, v3_resources_sources))"},{"line_number":106,"context_line":"    # TODO(ayoung): put token routes here"},{"line_number":107,"context_line":"    return wsgi.ComposingRouter(mapper, v3routers)"}],"source_content_type":"text/x-python","patch_set":1,"id":"baada198_923a0ce5","line":105,"updated":"2014-08-12 20:41:12.000000000","message":"this whole method is now really weird. using \"sources\" in the variable name just makes this all more confusing, and i don\u0027t see why it shouldn\u0027t just be \"_routers,\" except for the fact that there\u0027s already a v3routers variable in this method, and it would be even less clear how they\u0027re distinct. \u003c/confusing\u003e","commit_id":"eacc44fa549999464a03e0e7643f28d58a0cf231"},{"author":{"_account_id":6486,"name":"Brant Knudson","email":"blk@acm.org","username":"blk-u"},"change_message_id":"ae5798b177c3b5fd6995a21ba482242ec24dcd4e","unresolved":false,"context_lines":[{"line_number":102,"context_line":""},{"line_number":103,"context_line":"    # Add in the v3 version api"},{"line_number":104,"context_line":"    v3routers.append(routers.VersionV3(\u0027admin\u0027, v3_resources_sources))"},{"line_number":105,"context_line":"    v3routers.append(routers.VersionV3(\u0027public\u0027, v3_resources_sources))"},{"line_number":106,"context_line":"    # TODO(ayoung): put token routes here"},{"line_number":107,"context_line":"    return wsgi.ComposingRouter(mapper, v3routers)"}],"source_content_type":"text/x-python","patch_set":1,"id":"7ac7a964_4b4fe592","line":105,"in_reply_to":"baada198_923a0ce5","updated":"2014-08-14 01:53:49.000000000","message":"I changed the names... hopefully it\u0027s less confusing. I don\u0027t really understand it to begin with. I\u0027m willing to make whatever changes are necessary... would it make more sense to pass in a function to VersionV3 instead? Only one method out of the resources is required.","commit_id":"eacc44fa549999464a03e0e7643f28d58a0cf231"}]}