{"action":"created","issue":{"url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/76","labels_url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/76/labels{/name}","comments_url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/76/comments","events_url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/76/events","html_url":"https://github.com/swcarpentry/shell-novice/pull/76","id":57537086,"number":76,"title":"Update 04-loop.md","user":{"login":"DangarStu","id":10969948,"avatar_url":"https://avatars.githubusercontent.com/u/10969948?v=3","gravatar_id":"","url":"https://api.github.com/users/DangarStu","html_url":"https://github.com/DangarStu","followers_url":"https://api.github.com/users/DangarStu/followers","following_url":"https://api.github.com/users/DangarStu/following{/other_user}","gists_url":"https://api.github.com/users/DangarStu/gists{/gist_id}","starred_url":"https://api.github.com/users/DangarStu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DangarStu/subscriptions","organizations_url":"https://api.github.com/users/DangarStu/orgs","repos_url":"https://api.github.com/users/DangarStu/repos","events_url":"https://api.github.com/users/DangarStu/events{/privacy}","received_events_url":"https://api.github.com/users/DangarStu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-02-12T23:34:06Z","updated_at":"2015-02-13T00:16:36Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/swcarpentry/shell-novice/pulls/76","html_url":"https://github.com/swcarpentry/shell-novice/pull/76","diff_url":"https://github.com/swcarpentry/shell-novice/pull/76.diff","patch_url":"https://github.com/swcarpentry/shell-novice/pull/76.patch"},"body":"I feel this is an opportunity to reinforce how the shell goes through a pre-processing phase before calling any executables."},"comment":{"url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/comments/74181683","html_url":"https://github.com/swcarpentry/shell-novice/pull/76#issuecomment-74181683","issue_url":"https://api.github.com/repos/swcarpentry/shell-novice/issues/76","id":74181683,"user":{"login":"DangarStu","id":10969948,"avatar_url":"https://avatars.githubusercontent.com/u/10969948?v=3","gravatar_id":"","url":"https://api.github.com/users/DangarStu","html_url":"https://github.com/DangarStu","followers_url":"https://api.github.com/users/DangarStu/followers","following_url":"https://api.github.com/users/DangarStu/following{/other_user}","gists_url":"https://api.github.com/users/DangarStu/gists{/gist_id}","starred_url":"https://api.github.com/users/DangarStu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/DangarStu/subscriptions","organizations_url":"https://api.github.com/users/DangarStu/orgs","repos_url":"https://api.github.com/users/DangarStu/repos","events_url":"https://api.github.com/users/DangarStu/events{/privacy}","received_events_url":"https://api.github.com/users/DangarStu/received_events","type":"User","site_admin":false},"created_at":"2015-02-13T00:16:36Z","updated_at":"2015-02-13T00:16:36Z","body":"Sounds good to me. I agree that you can go too far and make it more\nconfusing for the student. :)\n\n*PARTING LINES*: \"A novel of human foibles and near redemption.\" – *Kirkus\nReviews*\n\n[image: Support independent publishing: Buy this book on Lulu.]\n<http://www.lulu.com/spotlight/dangarstu>\n\nhttp://dangarstu.com\n\nOn Fri, Feb 13, 2015 at 11:08 AM, W. Trevor King <notifications@github.com>\nwrote:\n\n> On Thu, Feb 12, 2015 at 03:34:08PM -0800, DangarStu wrote:\n> > I feel this is an opportunity to reinforce how the shell goes\n> > through a pre-processing phase before calling any executables.\n>\n> We need to walk the line between technical accuracy/detail and\n> accessibility. For example, you're introducing \"token\" here. Still,\n> I think explaining that this happens in the shell before calling the\n> command is worth the trouble. How about “The shell expands variables\n> before…”?\n>\n> —\n> Reply to this email directly or view it on GitHub\n> <https://github.com/swcarpentry/shell-novice/pull/76#issuecomment-74180745>\n> .\n>\n"}}