{"action":"opened","number":1990,"pull_request":{"url":"https://api.github.com/repos/cylc/cylc/pulls/1990","id":84882225,"html_url":"https://github.com/cylc/cylc/pull/1990","diff_url":"https://github.com/cylc/cylc/pull/1990.diff","patch_url":"https://github.com/cylc/cylc/pull/1990.patch","issue_url":"https://api.github.com/repos/cylc/cylc/issues/1990","number":1990,"state":"open","locked":false,"title":"cylc validate: fix demoted info concat","user":{"login":"matthewrmshin","id":908613,"avatar_url":"https://avatars.githubusercontent.com/u/908613?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewrmshin","html_url":"https://github.com/matthewrmshin","followers_url":"https://api.github.com/users/matthewrmshin/followers","following_url":"https://api.github.com/users/matthewrmshin/following{/other_user}","gists_url":"https://api.github.com/users/matthewrmshin/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewrmshin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewrmshin/subscriptions","organizations_url":"https://api.github.com/users/matthewrmshin/orgs","repos_url":"https://api.github.com/users/matthewrmshin/repos","events_url":"https://api.github.com/users/matthewrmshin/events{/privacy}","received_events_url":"https://api.github.com/users/matthewrmshin/received_events","type":"User","site_admin":false},"body":"Getting this trace back on a suite.\r\n\r\n```\r\nTraceback (most recent call last):\r\n  File \"/home/matt/cylc.git/bin/cylc-validate\", line 134, in <module>\r\n    main()\r\n  File \"/home/matt/cylc.git/bin/cylc-validate\", line 82, in main\r\n    mem_log_func=profiler.log_memory)\r\n  File \"/home/matt/cylc.git/lib/cylc/config.py\", line 134, in get_inst\r\n    vis_start_string, vis_stop_string, mem_log_func)\r\n  File \"/home/matt/cylc.git/lib/cylc/config.py\", line 340, in __init__\r\n    self.compute_family_tree()\r\n  File \"/home/matt/cylc.git/lib/cylc/config.py\", line 952, in compute_family_tree\r\n    log_msg += \" +\", p, \"as parent of '\" + n + \"'\"\r\nTypeError: cannot concatenate 'str' and 'tuple' objects\r\n```\r\n\r\nLooks like the `,` before and after `p` are turning right hand side into a tuple.\r\n\r\n@hjoliver please review.","created_at":"2016-09-12T14:33:44Z","updated_at":"2016-09-12T14:33:44Z","closed_at":null,"merged_at":null,"merge_commit_sha":null,"assignee":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"assignees":[{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false}],"milestone":{"url":"https://api.github.com/repos/cylc/cylc/milestones/53","html_url":"https://github.com/cylc/cylc/milestones/next%20release","labels_url":"https://api.github.com/repos/cylc/cylc/milestones/53/labels","id":1803480,"number":53,"title":"next release","description":"next release","creator":{"login":"hjoliver","id":2362137,"avatar_url":"https://avatars.githubusercontent.com/u/2362137?v=3","gravatar_id":"","url":"https://api.github.com/users/hjoliver","html_url":"https://github.com/hjoliver","followers_url":"https://api.github.com/users/hjoliver/followers","following_url":"https://api.github.com/users/hjoliver/following{/other_user}","gists_url":"https://api.github.com/users/hjoliver/gists{/gist_id}","starred_url":"https://api.github.com/users/hjoliver/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/hjoliver/subscriptions","organizations_url":"https://api.github.com/users/hjoliver/orgs","repos_url":"https://api.github.com/users/hjoliver/repos","events_url":"https://api.github.com/users/hjoliver/events{/privacy}","received_events_url":"https://api.github.com/users/hjoliver/received_events","type":"User","site_admin":false},"open_issues":1,"closed_issues":109,"state":"open","created_at":"2016-06-01T23:05:26Z","updated_at":"2016-09-12T14:33:44Z","due_on":"2016-09-12T00:00:00Z","closed_at":null},"commits_url":"https://api.github.com/repos/cylc/cylc/pulls/1990/commits","review_comments_url":"https://api.github.com/repos/cylc/cylc/pulls/1990/comments","review_comment_url":"https://api.github.com/repos/cylc/cylc/pulls/comments{/number}","comments_url":"https://api.github.com/repos/cylc/cylc/issues/1990/comments","statuses_url":"https://api.github.com/repos/cylc/cylc/statuses/c9ef60c78a8ba99050b55b100899e6f3450be615","head":{"label":"matthewrmshin:fix-concat","ref":"fix-concat","sha":"c9ef60c78a8ba99050b55b100899e6f3450be615","user":{"login":"matthewrmshin","id":908613,"avatar_url":"https://avatars.githubusercontent.com/u/908613?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewrmshin","html_url":"https://github.com/matthewrmshin","followers_url":"https://api.github.com/users/matthewrmshin/followers","following_url":"https://api.github.com/users/matthewrmshin/following{/other_user}","gists_url":"https://api.github.com/users/matthewrmshin/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewrmshin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewrmshin/subscriptions","organizations_url":"https://api.github.com/users/matthewrmshin/orgs","repos_url":"https://api.github.com/users/matthewrmshin/repos","events_url":"https://api.github.com/users/matthewrmshin/events{/privacy}","received_events_url":"https://api.github.com/users/matthewrmshin/received_events","type":"User","site_admin":false},"repo":{"id":4710539,"name":"cylc","full_name":"matthewrmshin/cylc","owner":{"login":"matthewrmshin","id":908613,"avatar_url":"https://avatars.githubusercontent.com/u/908613?v=3","gravatar_id":"","url":"https://api.github.com/users/matthewrmshin","html_url":"https://github.com/matthewrmshin","followers_url":"https://api.github.com/users/matthewrmshin/followers","following_url":"https://api.github.com/users/matthewrmshin/following{/other_user}","gists_url":"https://api.github.com/users/matthewrmshin/gists{/gist_id}","starred_url":"https://api.github.com/users/matthewrmshin/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/matthewrmshin/subscriptions","organizations_url":"https://api.github.com/users/matthewrmshin/orgs","repos_url":"https://api.github.com/users/matthewrmshin/repos","events_url":"https://api.github.com/users/matthewrmshin/events{/privacy}","received_events_url":"https://api.github.com/users/matthewrmshin/received_events","type":"User","site_admin":false},"private":false,"html_url":"https://github.com/matthewrmshin/cylc","description":"Cylc: an engine for running meteorological suites","fork":true,"url":"https://api.github.com/repos/matthewrmshin/cylc","forks_url":"https://api.github.com/repos/matthewrmshin/cylc/forks","keys_url":"https://api.github.com/repos/matthewrmshin/cylc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/matthewrmshin/cylc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/matthewrmshin/cylc/teams","hooks_url":"https://api.github.com/repos/matthewrmshin/cylc/hooks","issue_events_url":"https://api.github.com/repos/matthewrmshin/cylc/issues/events{/number}","events_url":"https://api.github.com/repos/matthewrmshin/cylc/events","assignees_url":"https://api.github.com/repos/matthewrmshin/cylc/assignees{/user}","branches_url":"https://api.github.com/repos/matthewrmshin/cylc/branches{/branch}","tags_url":"https://api.github.com/repos/matthewrmshin/cylc/tags","blobs_url":"https://api.github.com/repos/matthewrmshin/cylc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/matthewrmshin/cylc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/matthewrmshin/cylc/git/refs{/sha}","trees_url":"https://api.github.com/repos/matthewrmshin/cylc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/matthewrmshin/cylc/statuses/{sha}","languages_url":"https://api.github.com/repos/matthewrmshin/cylc/languages","stargazers_url":"https://api.github.com/repos/matthewrmshin/cylc/stargazers","contributors_url":"https://api.github.com/repos/matthewrmshin/cylc/contributors","subscribers_url":"https://api.github.com/repos/matthewrmshin/cylc/subscribers","subscription_url":"https://api.github.com/repos/matthewrmshin/cylc/subscription","commits_url":"https://api.github.com/repos/matthewrmshin/cylc/commits{/sha}","git_commits_url":"https://api.github.com/repos/matthewrmshin/cylc/git/commits{/sha}","comments_url":"https://api.github.com/repos/matthewrmshin/cylc/comments{/number}","issue_comment_url":"https://api.github.com/repos/matthewrmshin/cylc/issues/comments{/number}","contents_url":"https://api.github.com/repos/matthewrmshin/cylc/contents/{+path}","compare_url":"https://api.github.com/repos/matthewrmshin/cylc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/matthewrmshin/cylc/merges","archive_url":"https://api.github.com/repos/matthewrmshin/cylc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/matthewrmshin/cylc/downloads","issues_url":"https://api.github.com/repos/matthewrmshin/cylc/issues{/number}","pulls_url":"https://api.github.com/repos/matthewrmshin/cylc/pulls{/number}","milestones_url":"https://api.github.com/repos/matthewrmshin/cylc/milestones{/number}","notifications_url":"https://api.github.com/repos/matthewrmshin/cylc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/matthewrmshin/cylc/labels{/name}","releases_url":"https://api.github.com/repos/matthewrmshin/cylc/releases{/id}","deployments_url":"https://api.github.com/repos/matthewrmshin/cylc/deployments","created_at":"2012-06-19T07:07:09Z","updated_at":"2016-08-22T09:46:59Z","pushed_at":"2016-09-12T14:31:00Z","git_url":"git://github.com/matthewrmshin/cylc.git","ssh_url":"git@github.com:matthewrmshin/cylc.git","clone_url":"https://github.com/matthewrmshin/cylc.git","svn_url":"https://github.com/matthewrmshin/cylc","homepage":"http://cylc.github.io/cylc/","size":68911,"stargazers_count":2,"watchers_count":2,"language":"Python","has_issues":false,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"open_issues_count":0,"forks":0,"open_issues":0,"watchers":2,"default_branch":"master"}},"base":{"label":"cylc:master","ref":"master","sha":"3429079cc6c410087a94555a8c37ea2955085361","user":{"login":"cylc","id":824878,"avatar_url":"https://avatars.githubusercontent.com/u/824878?v=3","gravatar_id":"","url":"https://api.github.com/users/cylc","html_url":"https://github.com/cylc","followers_url":"https://api.github.com/users/cylc/followers","following_url":"https://api.github.com/users/cylc/following{/other_user}","gists_url":"https://api.github.com/users/cylc/gists{/gist_id}","starred_url":"https://api.github.com/users/cylc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cylc/subscriptions","organizations_url":"https://api.github.com/users/cylc/orgs","repos_url":"https://api.github.com/users/cylc/repos","events_url":"https://api.github.com/users/cylc/events{/privacy}","received_events_url":"https://api.github.com/users/cylc/received_events","type":"Organization","site_admin":false},"repo":{"id":1836229,"name":"cylc","full_name":"cylc/cylc","owner":{"login":"cylc","id":824878,"avatar_url":"https://avatars.githubusercontent.com/u/824878?v=3","gravatar_id":"","url":"https://api.github.com/users/cylc","html_url":"https://github.com/cylc","followers_url":"https://api.github.com/users/cylc/followers","following_url":"https://api.github.com/users/cylc/following{/other_user}","gists_url":"https://api.github.com/users/cylc/gists{/gist_id}","starred_url":"https://api.github.com/users/cylc/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cylc/subscriptions","organizations_url":"https://api.github.com/users/cylc/orgs","repos_url":"https://api.github.com/users/cylc/repos","events_url":"https://api.github.com/users/cylc/events{/privacy}","received_events_url":"https://api.github.com/users/cylc/received_events","type":"Organization","site_admin":false},"private":false,"html_url":"https://github.com/cylc/cylc","description":"Cylc, a workflow engine for cycling tasks.","fork":false,"url":"https://api.github.com/repos/cylc/cylc","forks_url":"https://api.github.com/repos/cylc/cylc/forks","keys_url":"https://api.github.com/repos/cylc/cylc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/cylc/cylc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/cylc/cylc/teams","hooks_url":"https://api.github.com/repos/cylc/cylc/hooks","issue_events_url":"https://api.github.com/repos/cylc/cylc/issues/events{/number}","events_url":"https://api.github.com/repos/cylc/cylc/events","assignees_url":"https://api.github.com/repos/cylc/cylc/assignees{/user}","branches_url":"https://api.github.com/repos/cylc/cylc/branches{/branch}","tags_url":"https://api.github.com/repos/cylc/cylc/tags","blobs_url":"https://api.github.com/repos/cylc/cylc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/cylc/cylc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/cylc/cylc/git/refs{/sha}","trees_url":"https://api.github.com/repos/cylc/cylc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/cylc/cylc/statuses/{sha}","languages_url":"https://api.github.com/repos/cylc/cylc/languages","stargazers_url":"https://api.github.com/repos/cylc/cylc/stargazers","contributors_url":"https://api.github.com/repos/cylc/cylc/contributors","subscribers_url":"https://api.github.com/repos/cylc/cylc/subscribers","subscription_url":"https://api.github.com/repos/cylc/cylc/subscription","commits_url":"https://api.github.com/repos/cylc/cylc/commits{/sha}","git_commits_url":"https://api.github.com/repos/cylc/cylc/git/commits{/sha}","comments_url":"https://api.github.com/repos/cylc/cylc/comments{/number}","issue_comment_url":"https://api.github.com/repos/cylc/cylc/issues/comments{/number}","contents_url":"https://api.github.com/repos/cylc/cylc/contents/{+path}","compare_url":"https://api.github.com/repos/cylc/cylc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/cylc/cylc/merges","archive_url":"https://api.github.com/repos/cylc/cylc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/cylc/cylc/downloads","issues_url":"https://api.github.com/repos/cylc/cylc/issues{/number}","pulls_url":"https://api.github.com/repos/cylc/cylc/pulls{/number}","milestones_url":"https://api.github.com/repos/cylc/cylc/milestones{/number}","notifications_url":"https://api.github.com/repos/cylc/cylc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/cylc/cylc/labels{/name}","releases_url":"https://api.github.com/repos/cylc/cylc/releases{/id}","deployments_url":"https://api.github.com/repos/cylc/cylc/deployments","created_at":"2011-06-02T08:31:49Z","updated_at":"2016-09-02T12:14:00Z","pushed_at":"2016-09-12T10:24:00Z","git_url":"git://github.com/cylc/cylc.git","ssh_url":"git@github.com:cylc/cylc.git","clone_url":"https://github.com/cylc/cylc.git","svn_url":"https://github.com/cylc/cylc","homepage":"https://cylc.github.io/cylc","size":95534,"stargazers_count":42,"watchers_count":42,"language":"Python","has_issues":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":24,"mirror_url":null,"open_issues_count":119,"forks":24,"open_issues":119,"watchers":42,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/cylc/cylc/pulls/1990"},"html":{"href":"https://github.com/cylc/cylc/pull/1990"},"issue":{"href":"https://api.github.com/repos/cylc/cylc/issues/1990"},"comments":{"href":"https://api.github.com/repos/cylc/cylc/issues/1990/comments"},"review_comments":{"href":"https://api.github.com/repos/cylc/cylc/pulls/1990/comments"},"review_comment":{"href":"https://api.github.com/repos/cylc/cylc/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/cylc/cylc/pulls/1990/commits"},"statuses":{"href":"https://api.github.com/repos/cylc/cylc/statuses/c9ef60c78a8ba99050b55b100899e6f3450be615"}},"merged":false,"mergeable":null,"mergeable_state":"unknown","merged_by":null,"comments":0,"review_comments":0,"commits":1,"additions":2,"deletions":2,"changed_files":1}}
{"actor":{"display_login":"matthewrmshin"}}