)]}'
{"nova/cmd/manage.py":[{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"878d17e9ae49938e0c276856e251ed273549f750","unresolved":false,"context_lines":[{"line_number":928,"context_line":"        \"\"\"Contract database schema.\"\"\""},{"line_number":929,"context_line":"        if force_experimental:"},{"line_number":930,"context_line":"            return migration.db_contract(dry_run)"},{"line_number":931,"context_line":"        print(\"Contract not run! Use --force-experimental to allow.\")"},{"line_number":932,"context_line":""},{"line_number":933,"context_line":"    def version(self):"},{"line_number":934,"context_line":"        \"\"\"Print the current database version.\"\"\""}],"source_content_type":"text/x-python","patch_set":1,"id":"fa32b979_794976b6","line":931,"updated":"2015-06-17 21:29:21.000000000","message":"I\u0027d suggest expanding this text some:\n\n    The \"contract\" command is experimental and potentially dangerous.\n    As such, it is disabled by default.  Enable using \"--force-experimental\".\n\nMight be a little overkill, I dunno.  If there\u0027s a reasonable way to ask for confirmation in nova-manage commands, you could also do something like:\n\n    if not force_experimental:\n        print(\u0027The \"contract\" command is experimental and potentially \u0027\n              \u0027dangerous.  Do you really mean to run it?\u0027)\n        result \u003d six.moves.input(\u0027y or n\u003e \u0027)\n        if result !\u003d \u0027y\u0027:\n            return None\n    return migration.db_contract(dry_run)","commit_id":"398e2aa9adbd6e0617307110941ec7bfec7c4243"},{"author":{"_account_id":679,"name":"Kevin L. Mitchell","email":"klmitch@mit.edu","username":"klmitch"},"change_message_id":"1c0ddf6994b9480664166344a694c333a18aef94","unresolved":false,"context_lines":[{"line_number":930,"context_line":"        if force_experimental:"},{"line_number":931,"context_line":"            return migration.db_contract(dry_run)"},{"line_number":932,"context_line":"        print(\u0027The \"contract\" command is experimental and potentially\u0027"},{"line_number":933,"context_line":"              \u0027dangerous. As such, it is disabled by default. Enable using\u0027"},{"line_number":934,"context_line":"              \u0027\"--force-experimental\".\u0027)"},{"line_number":935,"context_line":""},{"line_number":936,"context_line":"    def version(self):"}],"source_content_type":"text/x-python","patch_set":3,"id":"fa32b979_06b21ee5","line":933,"updated":"2015-06-18 15:32:33.000000000","message":"Needs some extra spaces, otherwise the output looks like:\n\n    The \"contract\" command is experimental and potentiallydangerous. As such, it is disabled by default. Enable using\"--force-experimental\".\n\nI recommend s/potentially\u0027/potentially \u0027/ and s/using\u0027/using \u0027/","commit_id":"22669761554fac4cffbff00a77e1a8f8ae670ffe"}]}