Comments on formative

[jhill1]

Nice work overall! Some nice bits of coding, e.g. the interactive() and some nice overall coding style. All tests pass and the code looks good and reads well.

Some fixes are:
1)

  if (!file.exists(TASK_FILE)) {
    #Listing expected output for test
    return("1. Item 1\n2. Item 2\n3. Item 3\n4. Item 4\n5. Item 5")
  }

Shouldn't be needed unless the tests are broken...

2. Commenting is necessary at times

3. The commits are a bit haphazard, working on e thing, then another and moving back to the first. It makes it quite hard to unpick, e.g. if you wanted to revert a commit, but not lose other changes.

4. You've broken the the --help on the arparser...

[jhill1]

@Nathaniel-Burford COmments above, click the wee speech bubble symbol for inline comments. Nice work.