Update patch set 1
Patch Set 1: Code-Review-1 (1 comment) Patch-set: 1 Reviewer: Gerrit User 12898 <12898@4a232e18-c5a9-48ee-94c0-e04e7cca6543> Label: Code-Review=-1
This commit is contained in:
parent
0bc3c05540
commit
1409214e7e
|
@ -0,0 +1,21 @@
|
|||
{
|
||||
"comments": [
|
||||
{
|
||||
"key": {
|
||||
"uuid": "dfbec78f_3944491b",
|
||||
"filename": "tox.ini",
|
||||
"patchSetId": 1
|
||||
},
|
||||
"lineNbr": 0,
|
||||
"author": {
|
||||
"id": 12898
|
||||
},
|
||||
"writtenOn": "2019-05-08T15:19:50Z",
|
||||
"side": 1,
|
||||
"message": "While this change is technically correct, and fine to merge. Overall it is the wrong direction. The URL on master should be either: https://releases.openstack.org/constraints/upper/train or https://releases.openstack.org/constraints/upper/master.\n\nAs a community we need to discuss which is the best option.\n\nPlease watch out for a discussion on openstack-discuss in the near future",
|
||||
"revId": "7d104100afc2c7a78634c7bde2f162277615ff2d",
|
||||
"serverId": "4a232e18-c5a9-48ee-94c0-e04e7cca6543",
|
||||
"unresolved": false
|
||||
}
|
||||
]
|
||||
}
|
Loading…
Reference in New Issue