payload: {"action":"created","comment":{"url":"https://api.github.com/repos/cylc/cylc/pulls/comments/256242309","pull_request_review_id":203023108,"id":256242309,"node_id":"MDI0OlB1bGxSZXF1ZXN0UmV2aWV3Q29tbWVudDI1NjI0MjMwOQ==","diff_hunk":"@@ -32,10 +32,12 @@ Arguments:\n                 valid module arguments (lower-case equivalents accepted).\n \"\"\"\n \n-import sys\n-import re\n import os\n-from subprocess import check_call, PIPE, Popen, CalledProcessError\n+import re\n+import sys\n+from subprocess import PIPE, CalledProcessError, check_call\n+\n+from cylc.sprocess import pcylc","path":"bin/cylc-check-software","position":12,"original_position":12,"commit_id":"cba3c2386386140ea9e3a7a5f8d1d2bf6eb0feec","original_commit_id":"cba3c2386386140ea9e3a7a5f8d1d2bf6eb0feec","user":{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false},"body":"using cylc_subproc and procopen works","created_at":"2019-02-13T04:15:02Z","updated_at":"2019-02-13T04:15:02Z","html_url":"https://github.com/cylc/cylc/pull/2949#discussion_r256242309","pull_request_url":"https://api.github.com/repos/cylc/cylc/pulls/2949","author_association":"NONE","_links":{"self":{"href":"https://api.github.com/repos/cylc/cylc/pulls/comments/256242309"},"html":{"href":"https://github.com/cylc/cylc/pull/2949#discussion_r256242309"},"pull_request":{"href":"https://api.github.com/repos/cylc/cylc/pulls/2949"}},"in_reply_to_id":255888795},"pull_request":{"url":"https://api.github.com/repos/cylc/cylc/pulls/2949","id":251803106,"node_id":"MDExOlB1bGxSZXF1ZXN0MjUxODAzMTA2","html_url":"https://github.com/cylc/cylc/pull/2949","diff_url":"https://github.com/cylc/cylc/pull/2949.diff","patch_url":"https://github.com/cylc/cylc/pull/2949.patch","issue_url":"https://api.github.com/repos/cylc/cylc/issues/2949","number":2949,"state":"open","locked":false,"title":"address some bandit security recommendations ","user":{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false},"body":"an addendum to the closed 2908 pull request","created_at":"2019-02-11T05:11:00Z","updated_at":"2019-02-13T04:15:02Z","closed_at":null,"merged_at":null,"merge_commit_sha":"7451b626bf8a97f3891ee9369128ecdceed2e73f","assignee":{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false},"assignees":[{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false}],"requested_reviewers":[],"requested_teams":[],"labels":[],"milestone":{"url":"https://api.github.com/repos/cylc/cylc/milestones/75","html_url":"https://github.com/cylc/cylc/milestone/75","labels_url":"https://api.github.com/repos/cylc/cylc/milestones/75/labels","id":4002029,"node_id":"MDk6TWlsZXN0b25lNDAwMjAyOQ==","number":75,"title":"next-release","description":"Maintenance release: cylc-7.8.2","creator":{"login":"hjoliver","id":2362137,"node_id":"MDQ6VXNlcjIzNjIxMzc=","avatar_url":"https://avatars0.githubusercontent.com/u/2362137?v=4","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":6,"closed_issues":9,"state":"open","created_at":"2019-01-25T11:01:38Z","updated_at":"2019-02-12T12:49:59Z","due_on":"2019-02-28T08:00:00Z","closed_at":null},"commits_url":"https://api.github.com/repos/cylc/cylc/pulls/2949/commits","review_comments_url":"https://api.github.com/repos/cylc/cylc/pulls/2949/comments","review_comment_url":"https://api.github.com/repos/cylc/cylc/pulls/comments{/number}","comments_url":"https://api.github.com/repos/cylc/cylc/issues/2949/comments","statuses_url":"https://api.github.com/repos/cylc/cylc/statuses/cba3c2386386140ea9e3a7a5f8d1d2bf6eb0feec","head":{"label":"MartinRyan:mryan-dev-001","ref":"mryan-dev-001","sha":"cba3c2386386140ea9e3a7a5f8d1d2bf6eb0feec","user":{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false},"repo":{"id":158337253,"node_id":"MDEwOlJlcG9zaXRvcnkxNTgzMzcyNTM=","name":"cylc","full_name":"MartinRyan/cylc","private":false,"owner":{"login":"MartinRyan","id":65905,"node_id":"MDQ6VXNlcjY1OTA1","avatar_url":"https://avatars3.githubusercontent.com/u/65905?v=4","gravatar_id":"","url":"https://api.github.com/users/MartinRyan","html_url":"https://github.com/MartinRyan","followers_url":"https://api.github.com/users/MartinRyan/followers","following_url":"https://api.github.com/users/MartinRyan/following{/other_user}","gists_url":"https://api.github.com/users/MartinRyan/gists{/gist_id}","starred_url":"https://api.github.com/users/MartinRyan/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/MartinRyan/subscriptions","organizations_url":"https://api.github.com/users/MartinRyan/orgs","repos_url":"https://api.github.com/users/MartinRyan/repos","events_url":"https://api.github.com/users/MartinRyan/events{/privacy}","received_events_url":"https://api.github.com/users/MartinRyan/received_events","type":"User","site_admin":false},"html_url":"https://github.com/MartinRyan/cylc","description":"Cylc, a workflow engine for cycling (and non-cycling) systems.","fork":true,"url":"https://api.github.com/repos/MartinRyan/cylc","forks_url":"https://api.github.com/repos/MartinRyan/cylc/forks","keys_url":"https://api.github.com/repos/MartinRyan/cylc/keys{/key_id}","collaborators_url":"https://api.github.com/repos/MartinRyan/cylc/collaborators{/collaborator}","teams_url":"https://api.github.com/repos/MartinRyan/cylc/teams","hooks_url":"https://api.github.com/repos/MartinRyan/cylc/hooks","issue_events_url":"https://api.github.com/repos/MartinRyan/cylc/issues/events{/number}","events_url":"https://api.github.com/repos/MartinRyan/cylc/events","assignees_url":"https://api.github.com/repos/MartinRyan/cylc/assignees{/user}","branches_url":"https://api.github.com/repos/MartinRyan/cylc/branches{/branch}","tags_url":"https://api.github.com/repos/MartinRyan/cylc/tags","blobs_url":"https://api.github.com/repos/MartinRyan/cylc/git/blobs{/sha}","git_tags_url":"https://api.github.com/repos/MartinRyan/cylc/git/tags{/sha}","git_refs_url":"https://api.github.com/repos/MartinRyan/cylc/git/refs{/sha}","trees_url":"https://api.github.com/repos/MartinRyan/cylc/git/trees{/sha}","statuses_url":"https://api.github.com/repos/MartinRyan/cylc/statuses/{sha}","languages_url":"https://api.github.com/repos/MartinRyan/cylc/languages","stargazers_url":"https://api.github.com/repos/MartinRyan/cylc/stargazers","contributors_url":"https://api.github.com/repos/MartinRyan/cylc/contributors","subscribers_url":"https://api.github.com/repos/MartinRyan/cylc/subscribers","subscription_url":"https://api.github.com/repos/MartinRyan/cylc/subscription","commits_url":"https://api.github.com/repos/MartinRyan/cylc/commits{/sha}","git_commits_url":"https://api.github.com/repos/MartinRyan/cylc/git/commits{/sha}","comments_url":"https://api.github.com/repos/MartinRyan/cylc/comments{/number}","issue_comment_url":"https://api.github.com/repos/MartinRyan/cylc/issues/comments{/number}","contents_url":"https://api.github.com/repos/MartinRyan/cylc/contents/{+path}","compare_url":"https://api.github.com/repos/MartinRyan/cylc/compare/{base}...{head}","merges_url":"https://api.github.com/repos/MartinRyan/cylc/merges","archive_url":"https://api.github.com/repos/MartinRyan/cylc/{archive_format}{/ref}","downloads_url":"https://api.github.com/repos/MartinRyan/cylc/downloads","issues_url":"https://api.github.com/repos/MartinRyan/cylc/issues{/number}","pulls_url":"https://api.github.com/repos/MartinRyan/cylc/pulls{/number}","milestones_url":"https://api.github.com/repos/MartinRyan/cylc/milestones{/number}","notifications_url":"https://api.github.com/repos/MartinRyan/cylc/notifications{?since,all,participating}","labels_url":"https://api.github.com/repos/MartinRyan/cylc/labels{/name}","releases_url":"https://api.github.com/repos/MartinRyan/cylc/releases{/id}","deployments_url":"https://api.github.com/repos/MartinRyan/cylc/deployments","created_at":"2018-11-20T05:47:27Z","updated_at":"2019-02-11T04:39:03Z","pushed_at":"2019-02-11T06:05:04Z","git_url":"git://github.com/MartinRyan/cylc.git","ssh_url":"git@github.com:MartinRyan/cylc.git","clone_url":"https://github.com/MartinRyan/cylc.git","svn_url":"https://github.com/MartinRyan/cylc","homepage":"https://cylc.github.io/cylc","size":138403,"stargazers_count":0,"watchers_count":0,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":false,"forks_count":0,"mirror_url":null,"archived":false,"open_issues_count":0,"license":{"key":"gpl-3.0","name":"GNU General Public License v3.0","spdx_id":"GPL-3.0","url":"https://api.github.com/licenses/gpl-3.0","node_id":"MDc6TGljZW5zZTk="},"forks":0,"open_issues":0,"watchers":0,"default_branch":"master"}},"base":{"label":"cylc:master","ref":"master","sha":"697c3c2ecbd7b34c2682caf74aae9206e6772450","user":{"login":"cylc","id":824878,"node_id":"MDEyOk9yZ2FuaXphdGlvbjgyNDg3OA==","avatar_url":"https://avatars1.githubusercontent.com/u/824878?v=4","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,"node_id":"MDEwOlJlcG9zaXRvcnkxODM2MjI5","name":"cylc","full_name":"cylc/cylc","private":false,"owner":{"login":"cylc","id":824878,"node_id":"MDEyOk9yZ2FuaXphdGlvbjgyNDg3OA==","avatar_url":"https://avatars1.githubusercontent.com/u/824878?v=4","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},"html_url":"https://github.com/cylc/cylc","description":"Cylc, a workflow engine for cycling (and non-cycling) systems.","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":"2019-02-11T03:57:09Z","pushed_at":"2019-02-12T12:50: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":136092,"stargazers_count":84,"watchers_count":84,"language":"Python","has_issues":true,"has_projects":true,"has_downloads":true,"has_wiki":true,"has_pages":true,"forks_count":57,"mirror_url":null,"archived":false,"open_issues_count":184,"license":{"key":"gpl-3.0","name":"GNU General Public License v3.0","spdx_id":"GPL-3.0","url":"https://api.github.com/licenses/gpl-3.0","node_id":"MDc6TGljZW5zZTk="},"forks":57,"open_issues":184,"watchers":84,"default_branch":"master"}},"_links":{"self":{"href":"https://api.github.com/repos/cylc/cylc/pulls/2949"},"html":{"href":"https://github.com/cylc/cylc/pull/2949"},"issue":{"href":"https://api.github.com/repos/cylc/cylc/issues/2949"},"comments":{"href":"https://api.github.com/repos/cylc/cylc/issues/2949/comments"},"review_comments":{"href":"https://api.github.com/repos/cylc/cylc/pulls/2949/comments"},"review_comment":{"href":"https://api.github.com/repos/cylc/cylc/pulls/comments{/number}"},"commits":{"href":"https://api.github.com/repos/cylc/cylc/pulls/2949/commits"},"statuses":{"href":"https://api.github.com/repos/cylc/cylc/statuses/cba3c2386386140ea9e3a7a5f8d1d2bf6eb0feec"}},"author_association":"NONE"}}
other: {"actor":{"display_login":"MartinRyan"}}