datacamp-content / courses-introduction-to-shell

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

Automated feedback on exercise 5.7 #123

Closed gvwilson closed 6 years ago

gvwilson commented 6 years ago

Exercise 5.7 | How can one shell script do many things?

Over the last 8 weeks, this exercise has received 16 feedback messages with 8 messages received over the last week. An automatically extracted summary of the issue is provided below

in range.sh, Total should be total

You can scan through the actual feedback messages to get more context on the. If this issue has already been addressed, please disregard it and close it.

created_at extra_info
2018-03-31 13:46:54 The original version of the range.sh file that the user is supposed to edit in the exercise contains the word 'Total'. However, the wc -l command contains the word 'total' (note the lower case 't'), so even if the user fills the placeholders correctly, his or her solution will not be taken as correct unless they also change 'Total' to 'total'.
2018-03-31 13:35:25 'Our shells scripts', which the exercise begins with, should probably read 'Our shell scripts'.
2018-03-31 00:16:24 Total should not be capitalized
2018-03-30 19:34:07 The base code has Total with a capital T and the solution has total with a lowercase t.
2018-03-29 15:19:19 'Total' is capitalized in the given script but needs to be lower case in the actual accepted solution. Also, you need to make clear that the solution has to be opened in the command shell under solutions.
2018-03-28 10:36:06 In the previous task, the given word "Total" should be replaced by "total"
2018-03-26 19:43:32 range.sh has "Total" instead of "total", but the instructions do not indicate that the existing code should be altered, only the blanks. The feedback for wrong answers is extremely unhelpful when the edited file is not what it expects and it took a bit of troubleshooting to find this error.
2018-03-26 14:41:40 in range.sh, Total should be total
2018-03-24 15:39:22 not "Total" but only "total"
2018-03-18 20:35:06 In step \#1, range.sh initially contains the following: "wc - l ____ | grep ____ Total" The word "Total" should not be capitalized and the user needs to lowercase it to get the code to work.
2018-03-18 11:42:25 Within the range.sh file the word "Total" is written with the uppercase hence the solution is not working properly until the lowercase is used
2018-03-08 22:11:22 should be a "total", not a "Total". "total" is in an output of wc -l !!!!!!!
2018-02-26 13:38:21 the 'Total' in the range.sh script should have a lower case, or the one in the solution should have a uppercase
2018-02-16 17:57:58 in range.sh, "Total" is spelled with a capital T by default. grep -v Total will not eliminate the line with the total in it because total is spelled with lower case "t." However, the instructions say to only modify the "___" text.
2018-02-13 13:24:28 The file range.sh contains word "Total" with capital letter, while the course expects "total" as the correct answer (as in range-1.sh).
2018-02-06 12:02:56 Total should be total without capital
gvwilson commented 6 years ago

Closed by #136.

gvwilson commented 6 years ago

Could someone please unassign this issue to me - thanks.