)]}'
{"oslo_policy/policy.py":[{"author":{"_account_id":28522,"name":"Hervé Beraud","email":"herveberaud.pro@gmail.com","username":"hberaud"},"change_message_id":"1b821e17976618b3faeeaf83675793fac391f52e","unresolved":false,"context_lines":[{"line_number":890,"context_line":"                # Check the scope of the operation against the possible scope"},{"line_number":891,"context_line":"                # attributes provided in `creds`."},{"line_number":892,"context_line":"                if creds.get(\u0027system\u0027):"},{"line_number":893,"context_line":"                    token_scope \u003d \u0027system\u0027  # nosec"},{"line_number":894,"context_line":"                else:"},{"line_number":895,"context_line":"                    # If the token isn\u0027t system-scoped then we\u0027re dealing with"},{"line_number":896,"context_line":"                    # either a domain-scoped token or a project-scoped token."}],"source_content_type":"text/x-python","patch_set":3,"id":"bf51134e_38cccc87","line":893,"range":{"start_line":893,"start_character":44,"end_line":893,"end_character":51},"updated":"2020-07-15 09:33:16.000000000","message":"I\u0027m not sure to seen why you added this comment, I suppose you want to ignore these lines during linters tests, but removing them doesn\u0027t change my local runs.\n\nAnyway, not a big deal, we are on rocky here and AFAIK this version is not a really active version so if CI is happy and if these rules help us to backport these changes I think that we can go ahead.","commit_id":"622bcf207948174535b4d04f15ef14fb265c834d"},{"author":{"_account_id":11904,"name":"Sean McGinnis","email":"sean.mcginnis@gmail.com","username":"SeanM"},"change_message_id":"29d3dbd899fce09c23035ab935d6ec8f3cfb97c9","unresolved":false,"context_lines":[{"line_number":890,"context_line":"                # Check the scope of the operation against the possible scope"},{"line_number":891,"context_line":"                # attributes provided in `creds`."},{"line_number":892,"context_line":"                if creds.get(\u0027system\u0027):"},{"line_number":893,"context_line":"                    token_scope \u003d \u0027system\u0027  # nosec"},{"line_number":894,"context_line":"                else:"},{"line_number":895,"context_line":"                    # If the token isn\u0027t system-scoped then we\u0027re dealing with"},{"line_number":896,"context_line":"                    # either a domain-scoped token or a project-scoped token."}],"source_content_type":"text/x-python","patch_set":3,"id":"9f560f44_5c299634","line":893,"range":{"start_line":893,"start_character":44,"end_line":893,"end_character":51},"in_reply_to":"bf51134e_38cccc87","updated":"2020-08-10 19:42:31.000000000","message":"pep8 error was probably that variable wasn\u0027t being set outside of conditional block. It probably would have been better to do something likeLl\n\ntoken_scope \u003d \u0027project\u0027\nif cred.get(\u0027system\u0027):\n    token_scope \u003d \u0027system\u0027\n\nBut too late for that now.","commit_id":"622bcf207948174535b4d04f15ef14fb265c834d"}]}