{"action":"created","issue":{"url":"https://api.github.com/repos/saltstack/salt/issues/27521","labels_url":"https://api.github.com/repos/saltstack/salt/issues/27521/labels{/name}","comments_url":"https://api.github.com/repos/saltstack/salt/issues/27521/comments","events_url":"https://api.github.com/repos/saltstack/salt/issues/27521/events","html_url":"https://github.com/saltstack/salt/pull/27521","id":108940536,"number":27521,"title":"Remove raw string formatting in execution modules","user":{"login":"terminalmage","id":328598,"avatar_url":"https://avatars.githubusercontent.com/u/328598?v=3","gravatar_id":"","url":"https://api.github.com/users/terminalmage","html_url":"https://github.com/terminalmage","followers_url":"https://api.github.com/users/terminalmage/followers","following_url":"https://api.github.com/users/terminalmage/following{/other_user}","gists_url":"https://api.github.com/users/terminalmage/gists{/gist_id}","starred_url":"https://api.github.com/users/terminalmage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/terminalmage/subscriptions","organizations_url":"https://api.github.com/users/terminalmage/orgs","repos_url":"https://api.github.com/users/terminalmage/repos","events_url":"https://api.github.com/users/terminalmage/events{/privacy}","received_events_url":"https://api.github.com/users/terminalmage/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/saltstack/salt/labels/Master%20Change","name":"Master Change","color":"0052cc"}],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-09-29T19:20:53Z","updated_at":"2015-10-12T16:20:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/saltstack/salt/pulls/27521","html_url":"https://github.com/saltstack/salt/pull/27521","diff_url":"https://github.com/saltstack/salt/pull/27521.diff","patch_url":"https://github.com/saltstack/salt/pull/27521.patch"},"body":"This pull request does a couple things:\r\n\r\n1. Removes raw string format flags\r\n2. Replaces, where found, instances where commands are passed as strings with lists of arguments. This should cut down on the amount of joining and splitting done, as well as significantly reduce the usage of ``str.format()``, giving us a bit of a performance boost.\r\n\r\nI expect a large number tests to fail at first, as it's quicker and easier for\r\nme to let jenkins sort it out than to try to run all the tests on the various\r\ndifferent platforms. A number of tests will likely need to be updated due to\r\nthe change of using lists of arguments instead of a string."},"comment":{"url":"https://api.github.com/repos/saltstack/salt/issues/comments/147450624","html_url":"https://github.com/saltstack/salt/pull/27521#issuecomment-147450624","issue_url":"https://api.github.com/repos/saltstack/salt/issues/27521","id":147450624,"user":{"login":"terminalmage","id":328598,"avatar_url":"https://avatars.githubusercontent.com/u/328598?v=3","gravatar_id":"","url":"https://api.github.com/users/terminalmage","html_url":"https://github.com/terminalmage","followers_url":"https://api.github.com/users/terminalmage/followers","following_url":"https://api.github.com/users/terminalmage/following{/other_user}","gists_url":"https://api.github.com/users/terminalmage/gists{/gist_id}","starred_url":"https://api.github.com/users/terminalmage/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/terminalmage/subscriptions","organizations_url":"https://api.github.com/users/terminalmage/orgs","repos_url":"https://api.github.com/users/terminalmage/repos","events_url":"https://api.github.com/users/terminalmage/events{/privacy}","received_events_url":"https://api.github.com/users/terminalmage/received_events","type":"User","site_admin":false},"created_at":"2015-10-12T16:20:08Z","updated_at":"2015-10-12T16:20:08Z","body":"Rebased."}}