Cleanup argument handling behaviors

[lukeed]

With #189 (thanks to @tjbenton!), fly.source, fly.clear, fly.start, and fly.watch had their APIs modified and/or refactored to allow for option objects to parsed from glob-arrays.

Despite passing tests, his first pass was unusable at runtime / outside of the test environment. (Add "better tests" to the todo list...)

This fills in the gaps while still maintaining the new features.

Changes:

• fly.start and fly.source lose "intelligent" globbing.

  Instead of passing an infinite argument list (with an options object as the last entry) for a spread-like behavior, there are now only two arguments: globs and options.

  This means that multiple glob entries must always be received as an array.

  Single glob selections may or may not choose to be an array.

  Examples:

  yield this.source("a.js")
  yield this.source("a.js", {ignore: "..."})
  yield this.source(["b.js"], {ignore: "..."})
  yield this.source(["a.js", "b.js", "c.js"], {ignore: "..."})
  // removed: this.source("a.js", "b.js", "c.js")
  
  yield this.start("task1")
  yield this.start("task1", {parallel: false})
  yield this.start(["task1", "task2"])
  yield this.start(["task1", "task2"], {parallel: true})
  // removed: this.start("task1", "task2")

• Update reporting for watch events
  • Improved performance (slightly) for logging/display logic.
  • Also changed format from (example) Changed ${filepath} to File changed: ${filepath}
• Update documentation to reflect changes
• Fix tests for API changes.
  • Remove duplicate test group.

---

Thanks to @hzlmn for pointing out the issue & jumpstarting the process.

[tjbenton]

What was the issue with rest arguments? I was actually working on adding more test cases for this while I was working on another feature.

[tjbenton]

I think I see why there was a problem, it comes from this commit. He updated the isPlainObject function to actually be correct because I had !== instead of === like it should have been. But when he made that change he didn't update the places where it was called to be !isPlainObject(options)

if (!isPlainObject(options)) {
    globs.push(options)
    options = {}
}

In the pr I'm working on I added 6+ tests cases fly.source and similar tests for the other functions that were touched.

[lukeed]

@tjbenton I did that as well, still had issues. 😟 I also checked out your PR commit directly.

It's okay though, I prefer this method of handling arguments. It's less code, more intuitive, and more consistent. Only fly.clear has the flexible sourcing (because it's one argument).