{"action":"created","issue":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/1303","repository_url":"https://api.github.com/repos/avocado-framework/avocado","labels_url":"https://api.github.com/repos/avocado-framework/avocado/issues/1303/labels{/name}","comments_url":"https://api.github.com/repos/avocado-framework/avocado/issues/1303/comments","events_url":"https://api.github.com/repos/avocado-framework/avocado/issues/1303/events","html_url":"https://github.com/avocado-framework/avocado/pull/1303","id":165423182,"number":1303,"title":"avocado/core/test.py: remove support for string test names and tags","user":{"login":"clebergnu","id":447178,"avatar_url":"https://avatars.githubusercontent.com/u/447178?v=3","gravatar_id":"","url":"https://api.github.com/users/clebergnu","html_url":"https://github.com/clebergnu","followers_url":"https://api.github.com/users/clebergnu/followers","following_url":"https://api.github.com/users/clebergnu/following{/other_user}","gists_url":"https://api.github.com/users/clebergnu/gists{/gist_id}","starred_url":"https://api.github.com/users/clebergnu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/clebergnu/subscriptions","organizations_url":"https://api.github.com/users/clebergnu/orgs","repos_url":"https://api.github.com/users/clebergnu/repos","events_url":"https://api.github.com/users/clebergnu/events{/privacy}","received_events_url":"https://api.github.com/users/clebergnu/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/avocado-framework/avocado/labels/in%20progress","name":"in progress","color":"ededed"}],"state":"open","locked":false,"assignee":{"login":"ldoktor","id":1074021,"avatar_url":"https://avatars.githubusercontent.com/u/1074021?v=3","gravatar_id":"","url":"https://api.github.com/users/ldoktor","html_url":"https://github.com/ldoktor","followers_url":"https://api.github.com/users/ldoktor/followers","following_url":"https://api.github.com/users/ldoktor/following{/other_user}","gists_url":"https://api.github.com/users/ldoktor/gists{/gist_id}","starred_url":"https://api.github.com/users/ldoktor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldoktor/subscriptions","organizations_url":"https://api.github.com/users/ldoktor/orgs","repos_url":"https://api.github.com/users/ldoktor/repos","events_url":"https://api.github.com/users/ldoktor/events{/privacy}","received_events_url":"https://api.github.com/users/ldoktor/received_events","type":"User","site_admin":false},"assignees":[{"login":"ldoktor","id":1074021,"avatar_url":"https://avatars.githubusercontent.com/u/1074021?v=3","gravatar_id":"","url":"https://api.github.com/users/ldoktor","html_url":"https://github.com/ldoktor","followers_url":"https://api.github.com/users/ldoktor/followers","following_url":"https://api.github.com/users/ldoktor/following{/other_user}","gists_url":"https://api.github.com/users/ldoktor/gists{/gist_id}","starred_url":"https://api.github.com/users/ldoktor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldoktor/subscriptions","organizations_url":"https://api.github.com/users/ldoktor/orgs","repos_url":"https://api.github.com/users/ldoktor/repos","events_url":"https://api.github.com/users/ldoktor/events{/privacy}","received_events_url":"https://api.github.com/users/ldoktor/received_events","type":"User","site_admin":false}],"milestone":null,"comments":3,"created_at":"2016-07-13T21:14:29Z","updated_at":"2016-07-23T10:03:19Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/avocado-framework/avocado/pulls/1303","html_url":"https://github.com/avocado-framework/avocado/pull/1303","diff_url":"https://github.com/avocado-framework/avocado/pull/1303.diff","patch_url":"https://github.com/avocado-framework/avocado/pull/1303.patch"},"body":"When the support for Test IDs was introduced, support for string based\r\ntest names and tags was kept in deprecated mode, scheduled to be\r\nremoved on an upcoming release.\r\n\r\nThis planned upcoming release for removal has already passed, so let's\r\nremove them now.\r\n\r\nReference: https://trello.com/c/QqZmfkYt\r\nSigned-off-by: Cleber Rosa <crosa@redhat.com>\r\n\r\n--\r\n\r\nThis has been tested with the following combination of versions:\r\n\r\n* Avocado 36lts (ab66fba) + Avocado-VT \"test_name_remove_deprecated\" (b82c0ac) = OK\r\n* Avocado master (c0ebd3f) + Avocado-VT \"test_name_remove_deprecated\" (b82c0ac) = OK\r\n* Avocado \"test_name_remove_deprecated\" (6baeeed) + Avocado-VT \"test_name_remove_deprecated\" (b82c0ac) = OK\r\n\r\nThe combination of Avocado \"test_name_remove_deprecated\" branch\r\n(6baeeed) + Avocado-VT master branch (1af13fd) does *not* work, as\r\nAvocado-VT users avocado.core.test.Test parameters that do not exist\r\nanymore.  But this is OK because Avocado-VT users are expected to\r\neither use Avocado from master or a LTS release.\r\n"},"comment":{"url":"https://api.github.com/repos/avocado-framework/avocado/issues/comments/234710418","html_url":"https://github.com/avocado-framework/avocado/pull/1303#issuecomment-234710418","issue_url":"https://api.github.com/repos/avocado-framework/avocado/issues/1303","id":234710418,"user":{"login":"ldoktor","id":1074021,"avatar_url":"https://avatars.githubusercontent.com/u/1074021?v=3","gravatar_id":"","url":"https://api.github.com/users/ldoktor","html_url":"https://github.com/ldoktor","followers_url":"https://api.github.com/users/ldoktor/followers","following_url":"https://api.github.com/users/ldoktor/following{/other_user}","gists_url":"https://api.github.com/users/ldoktor/gists{/gist_id}","starred_url":"https://api.github.com/users/ldoktor/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ldoktor/subscriptions","organizations_url":"https://api.github.com/users/ldoktor/orgs","repos_url":"https://api.github.com/users/ldoktor/repos","events_url":"https://api.github.com/users/ldoktor/events{/privacy}","received_events_url":"https://api.github.com/users/ldoktor/received_events","type":"User","site_admin":false},"created_at":"2016-07-23T10:03:19Z","updated_at":"2016-07-23T10:03:19Z","body":"OK, it looks quite good, there are many references to `tag` in documentation. Some should have been fixed earlier so we can simply create a card to update the docs, but some should be fixed within this PR, for example `docs/source/WritingTests.rst:914,40` as that would not work with those modifications."}}
{"actor":{"display_login":"ldoktor"}}