Possible 'strict' violations about backbone.advice HOT 4 CLOSED

Just tried pasting advice in to http://www.jshint.com/ and only warning I'm getting is that "define" is not declared. Can you shoot me the actual warnings? The jshint file I use to lint is included in the rep: https://github.com/rhysbrettbowen/Backbone.Advice/blob/master/.jshintrc so maybe just having that in the folder with advice will help

from backbone.advice.

Ahh this isn't necessarily about the source code of Advice itself, but rather implementing a mixin. Take for example this mixin:

define(function() {
   'use strict';

    function addMixin() {
        this.setDefaults({});
        this.after('initialize', function() {});
    }

    return addMixin;
});

Both times this is referenced we get a "Possible strict violation.". This makes sense since this ends up referring to something JSHint doesn't expect. I guess I could do something like following:

define(function() {
   'use strict';

    var mixin = {
        add: function() {
             this.setDefaults({});
             this.after('initialize', function() {});
        }
    }; 

    return mixin.add;
});

I was wondering if you've found a better way?

from backbone.advice.

Interestingly enough this seems to pass:

define(function() {
   'use strict';

    var namer = function() {
        this.setDefaults({});
        this.after('initialize', function() {});
    }

    return namer;
});

However, this can result in an unnamed function in older browsers :/. Maybe not so important.

from backbone.advice.

Closing.. ha

from backbone.advice.