)]}'
{"doc/requirements.txt":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"058748aced9de19919c94c272446b57e37994d74","unresolved":true,"context_lines":[{"line_number":7,"context_line":"# The below is rewquired to build testing module reference"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Include horizon as test requirement"},{"line_number":10,"context_line":"horizon\u003e\u003d17.1.0  # Apache-2.0"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"},{"line_number":13,"context_line":"xvfbwrapper\u003e\u003d0.1.3 #license: MIT"}],"source_content_type":"text/plain","patch_set":3,"id":"eada8bd3_ec8b5139","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":29},"updated":"2021-05-25 11:40:00.000000000","message":"I am not sure why horizon added as dependency for doc. at the first place.\nIMO you can remove horizon dependency from here.","commit_id":"258bd56effc2c3d7075049c5871fdfb435c5bac1"},{"author":{"_account_id":15519,"name":"Iury Gregory Melo Ferreira","display_name":"Iury Gregory","email":"iurygregory@gmail.com","username":"iurygregory"},"change_message_id":"354124a9e215e90a58657cabec902cf36f0c2eff","unresolved":true,"context_lines":[{"line_number":7,"context_line":"# The below is rewquired to build testing module reference"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Include horizon as test requirement"},{"line_number":10,"context_line":"horizon\u003e\u003d17.1.0  # Apache-2.0"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"},{"line_number":13,"context_line":"xvfbwrapper\u003e\u003d0.1.3 #license: MIT"}],"source_content_type":"text/plain","patch_set":3,"id":"b2bc520d_e8b17263","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":29},"in_reply_to":"174169fc_67faa9e1","updated":"2021-09-01 08:10:29.000000000","message":"Yeah, we probably don\u0027t need horizon as a doc dependency, I will push a patch for this","commit_id":"258bd56effc2c3d7075049c5871fdfb435c5bac1"},{"author":{"_account_id":33370,"name":"Hitesh Kumar","display_name":"HeroicHitesh","email":"email.hiteshkumar@gmail.com","username":"heroichitesh"},"change_message_id":"6fe2bc6a6eccf513d6b89a6e3c58980af07c9cfc","unresolved":true,"context_lines":[{"line_number":7,"context_line":"# The below is rewquired to build testing module reference"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"# Include horizon as test requirement"},{"line_number":10,"context_line":"horizon\u003e\u003d17.1.0  # Apache-2.0"},{"line_number":11,"context_line":""},{"line_number":12,"context_line":"selenium\u003e\u003d2.50.1 # Apache-2.0"},{"line_number":13,"context_line":"xvfbwrapper\u003e\u003d0.1.3 #license: MIT"}],"source_content_type":"text/plain","patch_set":3,"id":"174169fc_67faa9e1","line":10,"range":{"start_line":10,"start_character":0,"end_line":10,"end_character":29},"in_reply_to":"eada8bd3_ec8b5139","updated":"2021-05-25 13:00:27.000000000","message":"It\u0027s there since past 2 years and was added in this commit b68684a3ef6f75f65246066c28d1af879844741d","commit_id":"258bd56effc2c3d7075049c5871fdfb435c5bac1"}],"tox.ini":[{"author":{"_account_id":22873,"name":"Martin Kopec","email":"mkopec@redhat.com","username":"mkopec"},"change_message_id":"8e88eae674966a444c299d47124d66a83001af7f","unresolved":true,"context_lines":[{"line_number":54,"context_line":"[testenv:cover]"},{"line_number":55,"context_line":"commands \u003d"},{"line_number":56,"context_line":"  coverage erase"},{"line_number":57,"context_line":"  coverage run --source\u003dironic_ui {toxinidir}/manage.py test ironic_ui.test.tests --settings\u003dironic_ui.test.settings {posargs}"},{"line_number":58,"context_line":"  coverage xml"},{"line_number":59,"context_line":"  coverage html -d ./cover --omit\u003d\u0027*tests*\u0027"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"[testenv:docs]"},{"line_number":62,"context_line":"deps \u003d"}],"source_content_type":"text/x-properties","patch_set":2,"id":"a52d7566_e09acd73","side":"PARENT","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":43},"updated":"2021-05-21 12:56:01.000000000","message":"why were these changes necessary? there is not testr so no reason to put stestr there, plus it works fine as it is","commit_id":"c0e3256a20b253e07521ec33806222a27746af2f"},{"author":{"_account_id":33370,"name":"Hitesh Kumar","display_name":"HeroicHitesh","email":"email.hiteshkumar@gmail.com","username":"heroichitesh"},"change_message_id":"d0a9765387aa5bcd121d2b069d038520c2de58a1","unresolved":true,"context_lines":[{"line_number":54,"context_line":"[testenv:cover]"},{"line_number":55,"context_line":"commands \u003d"},{"line_number":56,"context_line":"  coverage erase"},{"line_number":57,"context_line":"  coverage run --source\u003dironic_ui {toxinidir}/manage.py test ironic_ui.test.tests --settings\u003dironic_ui.test.settings {posargs}"},{"line_number":58,"context_line":"  coverage xml"},{"line_number":59,"context_line":"  coverage html -d ./cover --omit\u003d\u0027*tests*\u0027"},{"line_number":60,"context_line":""},{"line_number":61,"context_line":"[testenv:docs]"},{"line_number":62,"context_line":"deps \u003d"}],"source_content_type":"text/x-properties","patch_set":2,"id":"a5a2a138_8a3e5687","side":"PARENT","line":59,"range":{"start_line":57,"start_character":0,"end_line":59,"end_character":43},"in_reply_to":"a52d7566_e09acd73","updated":"2021-05-21 13:17:52.000000000","message":"Sorry, my bad. I saw test written in coverage run command and thought it is testr but there is no --testr-args\nPS- After reverting it, cover works fine on my machine too.","commit_id":"c0e3256a20b253e07521ec33806222a27746af2f"}]}