)]}'
{"doc/source/dependency-management.rst":[{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions. Minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project. A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"},{"line_number":86,"context_line":"has dropped Python 2.7 support."},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"ced48691_fbc6a6e8","line":84,"range":{"start_line":82,"start_character":64,"end_line":84,"end_character":12},"updated":"2021-03-21 08:06:22.000000000","message":"I would move this sentence as the last paragraph in this section as it breaks the description of g-r (what follows is still about g-r, not l-c; that said, it\u0027s probably irrelevant to include anyhow as python2 is mostly gone).","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":true,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions. Minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project. A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"},{"line_number":86,"context_line":"has dropped Python 2.7 support."},{"line_number":87,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"acdabd61_74edf362","line":84,"range":{"start_line":82,"start_character":64,"end_line":84,"end_character":12},"in_reply_to":"ced48691_fbc6a6e8","updated":"2021-03-22 17:01:19.000000000","message":"It is extension to \"not setting minimum required versions\". let me put that in bracket to make it clear.","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":131,"context_line":"   or add constraints related to your addition."},{"line_number":132,"context_line":"2. Add the dependency to the appropriate requirements file(s) within"},{"line_number":133,"context_line":"   the project tree, providing a minimum version specifier. If the"},{"line_number":134,"context_line":"   ``lower-constraints.txt`` file exist in the project tree then update it"},{"line_number":135,"context_line":"   at the same time."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Removing a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"e074503a_d3837dad","line":134,"range":{"start_line":134,"start_character":59,"end_line":134,"end_character":60},"updated":"2021-03-21 08:06:22.000000000","message":"comma","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":131,"context_line":"   or add constraints related to your addition."},{"line_number":132,"context_line":"2. Add the dependency to the appropriate requirements file(s) within"},{"line_number":133,"context_line":"   the project tree, providing a minimum version specifier. If the"},{"line_number":134,"context_line":"   ``lower-constraints.txt`` file exist in the project tree then update it"},{"line_number":135,"context_line":"   at the same time."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Removing a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"94527e77_78138a46","line":134,"range":{"start_line":134,"start_character":34,"end_line":134,"end_character":40},"updated":"2021-03-21 08:06:22.000000000","message":"exists","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":131,"context_line":"   or add constraints related to your addition."},{"line_number":132,"context_line":"2. Add the dependency to the appropriate requirements file(s) within"},{"line_number":133,"context_line":"   the project tree, providing a minimum version specifier. If the"},{"line_number":134,"context_line":"   ``lower-constraints.txt`` file exist in the project tree then update it"},{"line_number":135,"context_line":"   at the same time."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Removing a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"7124b07b_5445e0d2","line":134,"range":{"start_line":134,"start_character":34,"end_line":134,"end_character":40},"in_reply_to":"94527e77_78138a46","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":131,"context_line":"   or add constraints related to your addition."},{"line_number":132,"context_line":"2. Add the dependency to the appropriate requirements file(s) within"},{"line_number":133,"context_line":"   the project tree, providing a minimum version specifier. If the"},{"line_number":134,"context_line":"   ``lower-constraints.txt`` file exist in the project tree then update it"},{"line_number":135,"context_line":"   at the same time."},{"line_number":136,"context_line":""},{"line_number":137,"context_line":"Removing a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"70d613ed_868507c0","line":134,"range":{"start_line":134,"start_character":59,"end_line":134,"end_character":60},"in_reply_to":"e074503a_d3837dad","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":150,"context_line":"   ``openstack/requirements``. If the version there is lower than the"},{"line_number":151,"context_line":"   desired version, prepare a patch to update the setting."},{"line_number":152,"context_line":"2. Update the minimum version in the relevant requirements file(s) in"},{"line_number":153,"context_line":"   the project tree. If the ``lower-constraints.txt`` file exist then"},{"line_number":154,"context_line":"   update it at the same patch."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Excluding a version of a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"9b0cba05_75c18148","line":153,"range":{"start_line":153,"start_character":59,"end_line":153,"end_character":64},"updated":"2021-03-21 08:06:22.000000000","message":"\"exists\" and then comma","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":150,"context_line":"   ``openstack/requirements``. If the version there is lower than the"},{"line_number":151,"context_line":"   desired version, prepare a patch to update the setting."},{"line_number":152,"context_line":"2. Update the minimum version in the relevant requirements file(s) in"},{"line_number":153,"context_line":"   the project tree. If the ``lower-constraints.txt`` file exist then"},{"line_number":154,"context_line":"   update it at the same patch."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Excluding a version of a dependency"}],"source_content_type":"text/x-rst","patch_set":4,"id":"505e0356_f88246ea","line":153,"range":{"start_line":153,"start_character":59,"end_line":153,"end_character":64},"in_reply_to":"9b0cba05_75c18148","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":151,"context_line":"   desired version, prepare a patch to update the setting."},{"line_number":152,"context_line":"2. Update the minimum version in the relevant requirements file(s) in"},{"line_number":153,"context_line":"   the project tree. If the ``lower-constraints.txt`` file exist then"},{"line_number":154,"context_line":"   update it at the same patch."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Excluding a version of a dependency"},{"line_number":157,"context_line":"-----------------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"77dd0854_52ea0c0f","line":154,"range":{"start_line":154,"start_character":13,"end_line":154,"end_character":16},"updated":"2021-03-21 08:06:22.000000000","message":"in","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":151,"context_line":"   desired version, prepare a patch to update the setting."},{"line_number":152,"context_line":"2. Update the minimum version in the relevant requirements file(s) in"},{"line_number":153,"context_line":"   the project tree. If the ``lower-constraints.txt`` file exist then"},{"line_number":154,"context_line":"   update it at the same patch."},{"line_number":155,"context_line":""},{"line_number":156,"context_line":"Excluding a version of a dependency"},{"line_number":157,"context_line":"-----------------------------------"}],"source_content_type":"text/x-rst","patch_set":4,"id":"8e66e268_a58280f2","line":154,"range":{"start_line":154,"start_character":13,"end_line":154,"end_character":16},"in_reply_to":"77dd0854_52ea0c0f","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"3a29e6a90b8770a79e5bf1ffe27979ddeae27478","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"fec056d7_dfa3541d","line":82,"updated":"2021-03-23 18:19:21.000000000","message":"I\u0027m replying to the same line of this patch as everyone else. Not sure what a thread is in this context (I suppose a new Gerrit feature which Gertty hasn\u0027t added support for?).","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"e05d5c615af8aecd465eaa62fb56dd7629c76932","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"397cf83a_c70ca5b1","line":82,"updated":"2021-03-23 01:03:39.000000000","message":"Oh, yep, I didn\u0027t even notice because it was already there, but worth fixing up in this change if the same general area of the document is already being touched. https://wiki.openstack.org/wiki/Diversity/Inclusivity suggests: denying, rejecting, excluding, blocking. We already use one of those words, so can probably just drop the parenthetical entirely in this situation.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":16708,"name":"Kendall Nelson","display_name":"Kendall (diablo_rojo)","email":"kennelson11@gmail.com","username":"kjnelson"},"change_message_id":"0b90eb06ed998bb5542d06b89c94aeec34958f5f","unresolved":true,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"7bf10c92_cb8bc99e","line":82,"range":{"start_line":82,"start_character":1,"end_line":82,"end_character":13},"updated":"2021-03-23 00:06:43.000000000","message":"Should probably remove this or get an alternative from the D\u0026I WG. I know its on the list of like.. words we want to avoid going forward because of its roots.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"0570ce6a7a1ee127116b179747496c88f0772613","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"8cee5968_9b16c658","line":82,"updated":"2021-03-23 17:00:19.000000000","message":"The query results weren\u0027t published because it was just informal research to try to see if we were on the right track with the terms identified. It was literally \"plug the words into codesearch.opendev.org and see what shows up\" anyway, nothing exciting in that regard. And I agree with Ghanshyam, cleanup via separate patches would also be entirely reasonable so as to not muddy the scope of unrelated content changes.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"af0867948bfcd234431e2c828e1b1922e4ae130b","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"2e12c611_8730bb24","line":82,"updated":"2021-03-23 16:05:28.000000000","message":"We did preliminary codesearch queries of various terms, including this one, mainly to determine the different contexts in which they were being used as well as rate at which they occur in our projects. That said, it\u0027s up to individual projects to decide what they\u0027re going to replace, with what, and when. The D\u0026I WG isn\u0027t going to do mass replacement patches, nor do they have the time even if they wanted to.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fd54cf4254afba44cd035225212d4a90e3071f16","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"01f1deb6_3a59b034","line":82,"in_reply_to":"2e12c611_8730bb24","updated":"2021-03-23 16:53:20.000000000","message":"Ack, that\u0027s reasonable. May I ask where to find the results? Or have the queries? I would check the impact on \"my projects\".","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"ba2ced2d5315ca1657e9596d6ac93d2c91dc0bf6","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"db8a994d_c52def5c","line":82,"in_reply_to":"2e12c611_8730bb24","updated":"2021-03-23 16:56:25.000000000","message":"I agree to fix those but disagree to mix up those fixes with others. We should do those cleanup separately as per their separate scope otherwise we can end up king every patch in most of the repo.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"ea33c1af36a0b836e15195ec1aa956722111cab5","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"f72090ed_2b748278","line":82,"in_reply_to":"3418ee53_6f9a1941","updated":"2021-03-23 17:15:01.000000000","message":"Yeah, I am aware but this is an edge case. First of all - this is not \"code\" in that sense - it does not necessarily \"run\" or will need to be bisected. Secondly, we already touch the offending line so the git diff will not be \"smaller\". We are also unlikely to revert this one. Anyhow, as I said, let\u0027s not waste more energy. Merge!","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"5208a72d4bbb6d579f7d53b3a71b513e7476a814","unresolved":true,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"d7a35f31_76fb301e","line":82,"range":{"start_line":82,"start_character":1,"end_line":82,"end_character":13},"in_reply_to":"7bf10c92_cb8bc99e","updated":"2021-03-23 13:53:47.000000000","message":"yeah, we can fix those things but that is separate thing to fix from this patch scope.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"9cc836b76edb863cb7a13f3fa8040ee758315ce1","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"22a066a1_3b03dac3","line":82,"in_reply_to":"8cee5968_9b16c658","updated":"2021-03-23 17:06:31.000000000","message":"You are somehow creating new threads each time you post a reply here.\nOK, I agree it\u0027s not worth wasting more energy on this patch. Let\u0027s merge it and think about D\u0026I in its own change.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"b7fec9925ca58ca540cae5139deaa399917bfbf3","unresolved":false,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"3418ee53_6f9a1941","line":82,"in_reply_to":"8cee5968_9b16c658","updated":"2021-03-23 17:07:57.000000000","message":"yeah basically this - https://wiki.openstack.org/wiki/GitCommitMessages#Structural_split_of_changes.\n\nmixing up it with other changes can leads to multiple issues as described in above page. Tempest has done it in a separate logical way and with separate new tag as there were interface changes also in Tempest.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"f9af4286e04f3ab1d440988632874fd53e501841","unresolved":true,"context_lines":[{"line_number":79,"context_line":"contents. Distributions may only be referenced by name, not URL. Options"},{"line_number":80,"context_line":"(such as -e or -f) may not be used. Environment markers"},{"line_number":81,"context_line":"and comments are permitted. Version specifiers are only allowed for excluding"},{"line_number":82,"context_line":"(blacklisting) versions, not setting minimum required versions (minimum"},{"line_number":83,"context_line":"required versions may optionally be specified in ``lower-constraints.txt``"},{"line_number":84,"context_line":"per-project). A single distribution may be listed more than once if different"},{"line_number":85,"context_line":"specifiers are required with different markers - for instance, if a dependency"}],"source_content_type":"text/x-rst","patch_set":5,"id":"5b76e2c8_72249020","line":82,"range":{"start_line":82,"start_character":1,"end_line":82,"end_character":13},"in_reply_to":"d7a35f31_76fb301e","updated":"2021-03-23 16:02:01.000000000","message":"I agree with Kendall that we can remove this one occurrence since we are touching this line. That said, the D\u0026I WG might want to run some codesearch queries to find these little nasties.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"}],"doc/source/testing.rst":[{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"6d61a31ff00481484e25c33487ae5a352fa01939","unresolved":false,"context_lines":[{"line_number":8,"context_line":"software that can be operated in a variety of configurations."},{"line_number":9,"context_line":"Manual localized testing is unlikely to be sufficient in this"},{"line_number":10,"context_line":"situation. By default all the testing is done with upper constraints"},{"line_number":11,"context_line":"of dependencies listed in ``upper-constraints.txt`` file and testing with"},{"line_number":12,"context_line":"lower constraints using ``lower-constraints.txt`` file is completely optional"},{"line_number":13,"context_line":"to the projects."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"1a1f409d_7ad5e3a4","line":11,"updated":"2021-03-19 20:00:49.000000000","message":"This could be mildly confusing since upper-constraints.txt is maintained centrally in openstack/requirements while lower-constraints.txt may exist within individual repositories. Maybe say:\n\n...with current versions of dependencies managed centrally in the ``upper-constraints.txt`` file within the ``requirements`` repository, while testing lower bounds of dependencies using a ``lower-constraints.txt`` within individual deliverable repositories is optional.","commit_id":"fcae947aaec2b6ed1704cffa7278b45dd4f8f826"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"92a7702431eb8556274ecd362c33b4f6b87984a4","unresolved":false,"context_lines":[{"line_number":8,"context_line":"software that can be operated in a variety of configurations."},{"line_number":9,"context_line":"Manual localized testing is unlikely to be sufficient in this"},{"line_number":10,"context_line":"situation. By default all the testing is done with upper constraints"},{"line_number":11,"context_line":"of dependencies listed in ``upper-constraints.txt`` file and testing with"},{"line_number":12,"context_line":"lower constraints using ``lower-constraints.txt`` file is completely optional"},{"line_number":13,"context_line":"to the projects."},{"line_number":14,"context_line":""}],"source_content_type":"text/x-rst","patch_set":1,"id":"2849925e_ab61af2b","line":11,"in_reply_to":"1a1f409d_7ad5e3a4","updated":"2021-03-19 20:07:06.000000000","message":"thanks, done","commit_id":"fcae947aaec2b6ed1704cffa7278b45dd4f8f826"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"01917b9e522f439eda6ff40a7a3576763ac3685a","unresolved":false,"context_lines":[{"line_number":11,"context_line":"in the ``upper-constraints.txt`` file within the ``openstack/requirements``"},{"line_number":12,"context_line":"repository, while testing lower bounds of dependencies using a"},{"line_number":13,"context_line":"``lower-constraints.txt`` within individual deliverable repositories"},{"line_number":14,"context_line":"is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"c71ed229_927e8584","line":14,"updated":"2021-03-19 20:44:27.000000000","message":"Close, my \"...\" meant don\u0027t drop the \"By default, all the testing is done\" part at the beginning. Sorry if that was vague.","commit_id":"e1806aa718360ba25bfd8cd837d909547869ada9"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"35fac0e839765448fbcf4a7ba15319bd1a305d6a","unresolved":false,"context_lines":[{"line_number":11,"context_line":"in the ``upper-constraints.txt`` file within the ``openstack/requirements``"},{"line_number":12,"context_line":"repository, while testing lower bounds of dependencies using a"},{"line_number":13,"context_line":"``lower-constraints.txt`` within individual deliverable repositories"},{"line_number":14,"context_line":"is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":3,"id":"7c7d01fc_09cf96cd","line":14,"in_reply_to":"c71ed229_927e8584","updated":"2021-03-20 00:34:50.000000000","message":"ah, got it my bad.","commit_id":"e1806aa718360ba25bfd8cd837d909547869ada9"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":9,"context_line":"Manual localized testing is unlikely to be sufficient in this"},{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"9ee425e9_4e980d15","line":12,"range":{"start_line":12,"start_character":53,"end_line":12,"end_character":60},"updated":"2021-03-21 08:06:22.000000000","message":"I would go with a full stop here.","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":9,"context_line":"Manual localized testing is unlikely to be sufficient in this"},{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""}],"source_content_type":"text/x-rst","patch_set":4,"id":"467ef6de_dc04a6ea","line":12,"range":{"start_line":12,"start_character":53,"end_line":12,"end_character":60},"in_reply_to":"9ee425e9_4e980d15","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"}],"source_content_type":"text/x-rst","patch_set":4,"id":"ffd6fc7b_4bc45d92","line":13,"range":{"start_line":13,"start_character":62,"end_line":13,"end_character":63},"updated":"2021-03-21 08:06:22.000000000","message":"file","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"}],"source_content_type":"text/x-rst","patch_set":4,"id":"4600de90_a83d739f","line":13,"range":{"start_line":13,"start_character":62,"end_line":13,"end_character":63},"in_reply_to":"ffd6fc7b_4bc45d92","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":30491,"name":"Radosław Piliszek","display_name":"Radek","email":"radek@piliszek.it","username":"yoctozepto","status":"self-employed techologist, collaborating mostly with 7bulls.com"},"change_message_id":"fb0add6190fd93df858bda36197de19cd1e4465b","unresolved":true,"context_lines":[{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"2352079a_a9db4a11","line":14,"range":{"start_line":13,"start_character":29,"end_line":14,"end_character":35},"updated":"2021-03-21 08:06:22.000000000","message":"I would surround this with commas for readability.","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":false,"context_lines":[{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"899e1b83_e1399e96","line":14,"range":{"start_line":13,"start_character":29,"end_line":14,"end_character":35},"in_reply_to":"2352079a_a9db4a11","updated":"2021-03-22 17:01:19.000000000","message":"Done","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":8556,"name":"Ghanshyam Maan","display_name":"Ghanshyam Maan","email":"gmaan.os14@gmail.com","username":"ghanshyam"},"change_message_id":"8ad48f1cf5a4af819e38df7702df883be00d13cd","unresolved":true,"context_lines":[{"line_number":10,"context_line":"situation. By default, all the testing is done with current versions"},{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository, while testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` within"},{"line_number":14,"context_line":"individual deliverable repositories is optional."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":4,"id":"4f312607_03d1ce75","line":14,"range":{"start_line":13,"start_character":29,"end_line":14,"end_character":35},"in_reply_to":"2352079a_a9db4a11","updated":"2021-03-22 17:01:19.000000000","message":"let me remove this to make it clear and add \u0027is optional for projects\u0027","commit_id":"7cc9b48680d68b57f6218e45058c0b52852c2a76"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"3a29e6a90b8770a79e5bf1ffe27979ddeae27478","unresolved":false,"context_lines":[{"line_number":11,"context_line":"of dependencies managed centrally in the ``upper-constraints.txt``"},{"line_number":12,"context_line":"file within the ``openstack/requirements`` repository. Testing"},{"line_number":13,"context_line":"lower bounds of dependencies using a ``lower-constraints.txt`` file"},{"line_number":14,"context_line":"is optional for projects."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Types of Tests"},{"line_number":17,"context_line":"\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d\u003d"}],"source_content_type":"text/x-rst","patch_set":5,"id":"896b40e2_573f96eb","line":14,"updated":"2021-03-23 18:19:21.000000000","message":"This doesn\u0027t get to my earlier point that lower-constraints.txt files are inside each project\u0027s deliverable repositories and not centrally managed (unlike upper-constraints.txt), but other reviewers have argued that being precise about that is going to confuse readers, so I guess I\u0027ll stop worrying about it.","commit_id":"ce850734efa0d3f448f59c2db598a4083ab6fbe4"}]}