Use @options (not @data) per Grunt guidelines

[jasonkarns]

To get target > task > default option merging out of the box, you have to
use options when configuring; as opposed to putting the settings directly
on the target. Then, within the task, use this.options([defaults]).

[searls]

Hehe, I got those guidelines into the readme because of their use of @DaTa and then I forgot to update this module. Nice

On Sun, Nov 10, 2013 at 3:57 PM, Jason Karns notifications@github.com
wrote:

To get target > task > default option merging out of the box, you have to
use options when configuring; as opposed to putting the settings directly
on the target. Then, within the task, use this.options([defaults]).
You can merge this Pull Request by running:
git pull https://github.com/jasonkarns/grunt-jasmine-bundle options
Or you can view, comment on it, or merge it online at:
#2
-- Commit Summary --

• Use @options (not @DaTa) per Grunt guidelines
• no need for _.extend; no need for _ at all
• de-dupe the specs array
• Update documentation and example with proper usage
• Correct grunt dependency
  -- File Changes --
  M README.md (20)
  M example/Gruntfile.js (6)
  M example/package.json (1)
  M package.json (1)
  M tasks/spec.coffee (21)
  -- Patch Links --
  https://github.com/testdouble/grunt-jasmine-bundle/pull/2.patch
  https://github.com/testdouble/grunt-jasmine-bundle/pull/2.diff

[jasonkarns]

Fair warning, this isn't backwards compatible. Any task configuration already using the old style won't get picked up correctly.

[jasonkarns]

I tried to tie this PR to #1, but apparently you can only submit PRs to issues created by you.

[searls]

Ah. We could make this BC easily enough

On Sun, Nov 10, 2013 at 4:00 PM, Jason Karns notifications@github.com
wrote:

Fair warning, this isn't backwards compatible. Any task configuration already using the old style won't get picked up correctly.

Reply to this email directly or view it on GitHub:
#2 (comment)

[jasonkarns]

Now backwards compatible. (and warns if using the old style config)

[davemo]

👍

[jasonkarns]

cleaned up the commits and plucked out the unrelated stuff.

ready to merge

[searls]

Merged on faith