datacamp-content / courses-introduction-to-shell

Introduction to Shell by Greg Wilson
Other
47 stars 47 forks source link

Instructions don't match solution | Chapter 5 Exercise 7 #98

Closed Hadrien-lcrx closed 6 years ago

Hadrien-lcrx commented 6 years ago

In How can one shell script do many things?, instructions state:

"Use Nano to edit the script range.sh and replace the ____ placeholders so that it lists the names and number of lines in all of the files given on the command line without showing the total number of lines in all files. (Do not try to subtract the column header lines from the files.)"

The solution is: cp /solutions/range-1.sh range.sh

This command copies the file, it doesn't edit anything. This is a similar issue to #97.

Recommendation

Update instructions, or update solution.

Screenshot

image

Cf faik.200@gmail.com, #63318 Cc @bakera81

Hadrien-lcrx commented 6 years ago

Another student had the same issue: "My Code is "Nano range.sh" and there "wc -l $@ | grep -i -v Total". It does what the exercise wants. It Counts the lines from each file and doesn't print the total over all files. I already took the hint which said to use "$@" so the first part should be correct. I also already used the solution which is "cp /solutions/range-1.sh range.sh" but I have no idea what to do with it."

Cf maierjannika@gmail.com, #64677 Cc @bakera81

gvwilson commented 6 years ago

Closing as a duplicate of #93 - basically, we can't run nano in solution blocks, so we have to do something else to get the build to succeed, and there's no way to run X for auto-testing but show Y to learners.

Karthi9934 commented 4 years ago

Facing similar issue have given feedback as well. Can you please update this solution