)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"cdafc37133369b7a6015a9e1024c03c3698f0a49","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Add flask as a dependency and basic setup."},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"    The setup_flask_app method creates `ironic_app` which is a basic"},{"line_number":10,"context_line":"    Flask app."},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"    If the attribute `flask_config` is True, then the class object \u0027Config\u0027 in"},{"line_number":13,"context_line":"    `config.py` will be used to set up the configuration."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"    See https://flask.palletsprojects.com/en/2.0.x/config/#configuration-basics"},{"line_number":16,"context_line":"    to learn more about the configuration."},{"line_number":17,"context_line":""},{"line_number":18,"context_line":"    config.py now have a class so the configuration is stored there."},{"line_number":19,"context_line":""},{"line_number":20,"context_line":"Change-Id: Ie8cbcda9239c8790cec900b0d4f3126888bbcea4"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"3eef168b_8b67bcd3","line":18,"range":{"start_line":9,"start_character":1,"end_line":18,"end_character":68},"updated":"2021-10-12 23:34:09.000000000","message":"Please don\u0027t indent this, and also wrap it \u003c80 characters. Preferably actually 68 characters, we have a number of contributors who use CLI code review clients.","commit_id":"900e80c5f6d284417f4a303d332a2eaca035260a"}],"ironic/api/app.py":[{"author":{"_account_id":11655,"name":"Julia Kreger","email":"juliaashleykreger@gmail.com","username":"jkreger","status":"Flying to the moon with a Jetpack!"},"change_message_id":"cdafc37133369b7a6015a9e1024c03c3698f0a49","unresolved":true,"context_lines":[{"line_number":14,"context_line":"#    WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied. See the"},{"line_number":15,"context_line":"#    License for the specific language governing permissions and limitations"},{"line_number":16,"context_line":"#    under the License."},{"line_number":17,"context_line":"from flask import Flask"},{"line_number":18,"context_line":""},{"line_number":19,"context_line":"from ironic_lib import auth_basic"},{"line_number":20,"context_line":"import keystonemiddleware.audit as audit_middleware"}],"source_content_type":"text/x-python","patch_set":1,"id":"19d26cfe_f9e2140f","line":17,"updated":"2021-10-12 23:34:09.000000000","message":"Openstack has a particular order of imports. That is what the Pep8 I102 error is trying to suggest.\n\nThe order is:\n\nBuiltin imports from python\n\n3rd party imports (keep in mind, alphabetcial order)\n\nin-package imports.\n\n\nCheckout https://docs.openstack.org/hacking/latest/user/hacking.html","commit_id":"900e80c5f6d284417f4a303d332a2eaca035260a"}]}