)]}'
{"CONTRIBUTING.md":[{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"d4014d3ab1544d57affa02a63d18611dd93334c6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3fa7e38b_865ba6f4","updated":"2019-10-22 19:30:25.000000000","message":"I think this file would benefit from line wrapping","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"3fa7e38b_8e668355","in_reply_to":"3fa7e38b_865ba6f4","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"## Support Channels"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Whether you are a user or contributor, official support channels include:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- [Bug Reports](https://airship.atlassian.net/issues/?jql\u003dproject%20%3D%20AIR%20AND%20issuetype%20%3D%20Bug%20order%20by%20created%20DESC)"},{"line_number":10,"context_line":"- IRC:"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_7ddb904e","line":7,"range":{"start_line":7,"start_character":39,"end_line":7,"end_character":64},"updated":"2019-10-23 08:11:50.000000000","message":"Here is a list of support/contact channels: https://wiki.openstack.org/wiki/Airship#Get_in_Touch","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":4,"context_line":""},{"line_number":5,"context_line":"## Support Channels"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Whether you are a user or contributor, official support channels include:"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- [Bug Reports](https://airship.atlassian.net/issues/?jql\u003dproject%20%3D%20AIR%20AND%20issuetype%20%3D%20Bug%20order%20by%20created%20DESC)"},{"line_number":10,"context_line":"- IRC:"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_4ede4bdb","line":7,"range":{"start_line":7,"start_character":39,"end_line":7,"end_character":64},"in_reply_to":"3fa7e38b_7ddb904e","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- [Bug Reports](https://airship.atlassian.net/issues/?jql\u003dproject%20%3D%20AIR%20AND%20issuetype%20%3D%20Bug%20order%20by%20created%20DESC)"},{"line_number":10,"context_line":"- IRC:"},{"line_number":11,"context_line":"  - User: #airshipit"},{"line_number":12,"context_line":"  - Contributor: #airshipit"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Before opening a new issue or submitting a patchset, it\u0027s helpful to search the project - it\u0027s likely that another user has already"},{"line_number":15,"context_line":"reported the issue you\u0027re facing, or it\u0027s a known issue that we\u0027re already aware of. It is also worth asking on the IRC channels."}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_1d15bce5","line":12,"range":{"start_line":11,"start_character":4,"end_line":12,"end_character":15},"updated":"2019-10-23 08:11:50.000000000","message":"User and contributor? Should probably be Channel and IRC network.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"- [Bug Reports](https://airship.atlassian.net/issues/?jql\u003dproject%20%3D%20AIR%20AND%20issuetype%20%3D%20Bug%20order%20by%20created%20DESC)"},{"line_number":10,"context_line":"- IRC:"},{"line_number":11,"context_line":"  - User: #airshipit"},{"line_number":12,"context_line":"  - Contributor: #airshipit"},{"line_number":13,"context_line":""},{"line_number":14,"context_line":"Before opening a new issue or submitting a patchset, it\u0027s helpful to search the project - it\u0027s likely that another user has already"},{"line_number":15,"context_line":"reported the issue you\u0027re facing, or it\u0027s a known issue that we\u0027re already aware of. It is also worth asking on the IRC channels."}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_aee6dfaf","line":12,"range":{"start_line":11,"start_character":4,"end_line":12,"end_character":15},"in_reply_to":"3fa7e38b_1d15bce5","updated":"2019-11-07 18:47:03.000000000","message":"Done by effectively linking to the wiki contact and removing this.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The airshipctl project leverages 1-month sprints primarily for the purpose of chronologically ordering work in Jira."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"### Coding Conventions"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Coding conventions and standards are explained in the official developer docs:"},{"line_number":33,"context_line":"[Developers Guide](docs/developers.md)"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_9db76cf6","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":22},"updated":"2019-10-23 08:11:50.000000000","message":"Should be reference to https://airship-docs.readthedocs.io/en/latest/conventions.html.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":27,"context_line":""},{"line_number":28,"context_line":"The airshipctl project leverages 1-month sprints primarily for the purpose of chronologically ordering work in Jira."},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"### Coding Conventions"},{"line_number":31,"context_line":""},{"line_number":32,"context_line":"Coding conventions and standards are explained in the official developer docs:"},{"line_number":33,"context_line":"[Developers Guide](docs/developers.md)"}],"source_content_type":"text/x-gfm","patch_set":4,"id":"3fa7e38b_aeb49f83","line":30,"range":{"start_line":30,"start_character":0,"end_line":30,"end_character":22},"in_reply_to":"3fa7e38b_9db76cf6","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"}],"docs/developers.md":[{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":24,"context_line":"This will build the airshipctl binary."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"To run all the tests including linting and coverage reports, run"},{"line_number":27,"context_line":"`make test`. To run all tests in a containerized environment, run `make"},{"line_number":28,"context_line":"docker-image-unit-tests` or `make docker-image-lint`"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"To run airshipctl locally, you can run `bin/airshipctl`."}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_3d93d85e","line":27,"range":{"start_line":27,"start_character":6,"end_line":27,"end_character":10},"updated":"2019-10-23 08:11:50.000000000","message":"`tests`, as per https://airship-docs.readthedocs.io/en/latest/code-conventions.html#makefile","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":24,"context_line":"This will build the airshipctl binary."},{"line_number":25,"context_line":""},{"line_number":26,"context_line":"To run all the tests including linting and coverage reports, run"},{"line_number":27,"context_line":"`make test`. To run all tests in a containerized environment, run `make"},{"line_number":28,"context_line":"docker-image-unit-tests` or `make docker-image-lint`"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"To run airshipctl locally, you can run `bin/airshipctl`."}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_0e4e1396","line":27,"range":{"start_line":27,"start_character":6,"end_line":27,"end_character":10},"in_reply_to":"3fa7e38b_3d93d85e","updated":"2019-11-07 18:47:03.000000000","message":"I am going to document the current functionality as-is.  If we want to change it, we need to change the makefile.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"To run all the tests including linting and coverage reports, run"},{"line_number":27,"context_line":"`make test`. To run all tests in a containerized environment, run `make"},{"line_number":28,"context_line":"docker-image-unit-tests` or `make docker-image-lint`"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"To run airshipctl locally, you can run `bin/airshipctl`."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_bd1268b3","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":6},"updated":"2019-10-23 08:11:50.000000000","message":"I would suggest to avoid using `docker`, and use non-vendor specific language in relation to containers wherever possible. It would help to those who are using podman, skopeo, etc. etc.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":25,"context_line":""},{"line_number":26,"context_line":"To run all the tests including linting and coverage reports, run"},{"line_number":27,"context_line":"`make test`. To run all tests in a containerized environment, run `make"},{"line_number":28,"context_line":"docker-image-unit-tests` or `make docker-image-lint`"},{"line_number":29,"context_line":""},{"line_number":30,"context_line":"To run airshipctl locally, you can run `bin/airshipctl`."},{"line_number":31,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_ee48578b","line":28,"range":{"start_line":28,"start_character":0,"end_line":28,"end_character":6},"in_reply_to":"3fa7e38b_bd1268b3","updated":"2019-11-07 18:47:03.000000000","message":"I am going to document the current functionality as-is.  If we want to change it, we need to change the makefile.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":33,"context_line":""},{"line_number":34,"context_line":"If you want to build an `airshipctl` Docker container, run `make docker-image`."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"To build Docker images, use `make docker-build`."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Pre-build images are already available at [quay.io](http://quay.io/airshipit/airshipctl)."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_dd8ba41b","line":36,"range":{"start_line":36,"start_character":34,"end_line":36,"end_character":46},"updated":"2019-10-23 08:11:50.000000000","message":"`images` as per https://airship-docs.readthedocs.io/en/latest/code-conventions.html#makefile","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"d4014d3ab1544d57affa02a63d18611dd93334c6","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"## Docker Images"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"If you want to build an `airshipctl` Docker container, run `make docker-image`."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"To build Docker images, use `make docker-build`."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Pre-build images are already available at [quay.io](http://quay.io/airshipit/airshipctl)."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_5539d6ef","line":36,"range":{"start_line":34,"start_character":0,"end_line":36,"end_character":48},"updated":"2019-10-22 19:30:25.000000000","message":"`make docker-image` creates a Docker image.\n`make docker-build` doesn\u0027t exist.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":31,"context_line":""},{"line_number":32,"context_line":"## Docker Images"},{"line_number":33,"context_line":""},{"line_number":34,"context_line":"If you want to build an `airshipctl` Docker container, run `make docker-image`."},{"line_number":35,"context_line":""},{"line_number":36,"context_line":"To build Docker images, use `make docker-build`."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Pre-build images are already available at [quay.io](http://quay.io/airshipit/airshipctl)."},{"line_number":39,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_2e7d0fb2","line":36,"range":{"start_line":34,"start_character":0,"end_line":36,"end_character":48},"in_reply_to":"3fa7e38b_5539d6ef","updated":"2019-11-07 18:47:03.000000000","message":"I am going to document the current functionality as-is.  If we want to change it, we need to change the makefile.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"d4014d3ab1544d57affa02a63d18611dd93334c6","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"To build Docker images, use `make docker-build`."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Pre-build images are already available at [quay.io](http://quay.io/airshipit/airshipctl)."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"## Contribution Guidelines"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_351e9a74","line":38,"range":{"start_line":38,"start_character":4,"end_line":38,"end_character":9},"updated":"2019-10-22 19:30:25.000000000","message":"built","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":35,"context_line":""},{"line_number":36,"context_line":"To build Docker images, use `make docker-build`."},{"line_number":37,"context_line":""},{"line_number":38,"context_line":"Pre-build images are already available at [quay.io](http://quay.io/airshipit/airshipctl)."},{"line_number":39,"context_line":""},{"line_number":40,"context_line":"## Contribution Guidelines"},{"line_number":41,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_6e7307ea","line":38,"range":{"start_line":38,"start_character":4,"end_line":38,"end_character":9},"in_reply_to":"3fa7e38b_351e9a74","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Make sure you have read and understood the main CONTRIBUTING guide:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"https://github.com/airshipit/airshipctl/blob/master/CONTRIBUTING.md"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"### Structure of the Code"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_9d852c0a","line":50,"range":{"start_line":50,"start_character":8,"end_line":50,"end_character":18},"updated":"2019-10-23 08:11:50.000000000","message":"Should be opendev.org URL as the primary. Github is a mirror.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":47,"context_line":""},{"line_number":48,"context_line":"Make sure you have read and understood the main CONTRIBUTING guide:"},{"line_number":49,"context_line":""},{"line_number":50,"context_line":"https://github.com/airshipit/airshipctl/blob/master/CONTRIBUTING.md"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"### Structure of the Code"},{"line_number":53,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_ce479b77","line":50,"range":{"start_line":50,"start_character":8,"end_line":50,"end_character":18},"in_reply_to":"3fa7e38b_9d852c0a","updated":"2019-11-07 18:47:03.000000000","message":"I have changed this to a relative URL.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7185,"name":"Dmitry Ukov","email":"dukov@mirantis.com","username":"dukov"},"change_message_id":"5dc709665b4741a8468d4c23a8108183f9ee3032","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"https://github.com/airshipit/airshipctl/blob/master/CONTRIBUTING.md"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"### Structure of the Code"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"The code for the airshipctl project is organized as follows:"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_0de69517","line":52,"range":{"start_line":52,"start_character":0,"end_line":52,"end_character":25},"updated":"2019-10-29 11:42:42.000000000","message":"I guess we may need to add link to unit testing guidelines defined in https://review.opendev.org/#/c/686758/","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":49,"context_line":""},{"line_number":50,"context_line":"https://github.com/airshipit/airshipctl/blob/master/CONTRIBUTING.md"},{"line_number":51,"context_line":""},{"line_number":52,"context_line":"### Structure of the Code"},{"line_number":53,"context_line":""},{"line_number":54,"context_line":"The code for the airshipctl project is organized as follows:"},{"line_number":55,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_4e53ab3a","line":52,"range":{"start_line":52,"start_character":0,"end_line":52,"end_character":25},"in_reply_to":"3fa7e38b_0de69517","updated":"2019-11-07 18:47:03.000000000","message":"I link to this document in the testing section below.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28208,"name":"Roman Gorshunov","email":"roman.gorshunov@att.com","username":"gorshunovr"},"change_message_id":"f556e0b4731bf87835a644eb6f6f5789c20b88b0","unresolved":false,"context_lines":[{"line_number":75,"context_line":"workflow for doing this is as follows:"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"1. `git clone` the"},{"line_number":78,"context_line":"   `github.com/airshipit/airshipctl` repository."},{"line_number":79,"context_line":"2. Create a new working branch (`git checkout -b feat/my-feature`) and"},{"line_number":80,"context_line":"   do your work on that branch."},{"line_number":81,"context_line":"3. When you are ready for us to review, push your branch to gerrit using `git-review`.  For"}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_5d77b43c","line":78,"range":{"start_line":78,"start_character":4,"end_line":78,"end_character":14},"updated":"2019-10-23 08:11:50.000000000","message":"Should be opendev.org URL as the primary. Github is a mirror.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":75,"context_line":"workflow for doing this is as follows:"},{"line_number":76,"context_line":""},{"line_number":77,"context_line":"1. `git clone` the"},{"line_number":78,"context_line":"   `github.com/airshipit/airshipctl` repository."},{"line_number":79,"context_line":"2. Create a new working branch (`git checkout -b feat/my-feature`) and"},{"line_number":80,"context_line":"   do your work on that branch."},{"line_number":81,"context_line":"3. When you are ready for us to review, push your branch to gerrit using `git-review`.  For"}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_0e337315","line":78,"range":{"start_line":78,"start_character":4,"end_line":78,"end_character":14},"in_reply_to":"3fa7e38b_5d77b43c","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":28869,"name":"Ian Howell","email":"ian.howell0@gmail.com","username":"ian"},"change_message_id":"d4014d3ab1544d57affa02a63d18611dd93334c6","unresolved":false,"context_lines":[{"line_number":86,"context_line":"We follow the Go coding style standards very closely. Typically, running"},{"line_number":87,"context_line":"`go fmt` will make your code beautiful for you."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"We also typically follow the conventions recommended by `go lint`."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Read more:"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_c000a603","line":89,"range":{"start_line":89,"start_character":0,"end_line":89,"end_character":66},"updated":"2019-10-22 19:30:25.000000000","message":"This technically isn\u0027t true, we don\u0027t use `go lint` at all. It might be worth mentioning that we use `golangci-lint` though.","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"},{"author":{"_account_id":7769,"name":"Pentheus","display_name":"Alan Meadows","email":"alan.meadows@gmail.com","username":"alanmeadows"},"change_message_id":"096a266ebf44b942be945a72e1a511240af986cf","unresolved":false,"context_lines":[{"line_number":86,"context_line":"We follow the Go coding style standards very closely. Typically, running"},{"line_number":87,"context_line":"`go fmt` will make your code beautiful for you."},{"line_number":88,"context_line":""},{"line_number":89,"context_line":"We also typically follow the conventions recommended by `go lint`."},{"line_number":90,"context_line":""},{"line_number":91,"context_line":"Read more:"},{"line_number":92,"context_line":""}],"source_content_type":"text/x-markdown","patch_set":4,"id":"3fa7e38b_8e1e8372","line":89,"range":{"start_line":89,"start_character":0,"end_line":89,"end_character":66},"in_reply_to":"3fa7e38b_c000a603","updated":"2019-11-07 18:47:03.000000000","message":"Done","commit_id":"34a6484fba3c488285f1c2843e6589a37dd0b5d5"}]}