Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Suggested Loops update #971

Open
mozeran opened this issue May 15, 2019 · 1 comment

Comments

Projects
None yet
2 participants
@mozeran
Copy link

commented May 15, 2019

In the Loops episode, the description of the first example is incorrect. (I'm guessing the example was previously modified but the explanation was not.) Here's what it says now:

The interpreter runs the command head on basilisk.dat, and then prints the first three lines of basilisk.dat. For the second iteration, $filename becomes unicorn.dat. This time, the shell runs head on unicorn.dat and prints the first three lines of unicorn.dat.

This is what I would suggest changing it to:

The interpreter runs the command head on basilisk.dat, pipes the first two lines to the tail command, and then prints the second line of basilisk.dat. For the second iteration, $filename becomes unicorn.dat. This time, the shell runs head on unicorn.dat, pipes the first two lines to the tail command, and then prints the second line of unicorn.dat.

@colinmorris

This comment has been minimized.

Copy link
Contributor

commented May 21, 2019

Nice catch. Your proposed rewording looks good - PR welcome!

mozeran added a commit to mozeran/shell-novice that referenced this issue May 21, 2019

Update example explanation
Pull request to solve issue 971 swcarpentry#971
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.