NickHeiner

> [email protected] test /Users/nick.heiner/public-projects/npm
> node ./test/run.js && tap test/tap/*.js

# testing in /var/folders/6v/3scp840j6gzgwzp0ctz_xx_40000gp/T/npm-test-38506
# global prefix = /var/folders/6v/3scp840j6gzgwzp0ctz_xx_40000gp/T/npm-test-38506/root
ok 1 node "/Users/nick.heiner/public-projects/npm/bin/npm-cli.js" install "/Users/nick.heiner/public-projects/npm"
ok 2 npm config set package-config:foo boo
ok 3 npm install /Users/nick.heiner/public-projects/npm
ok 4 npm install packages/npm-test-array-bin

NickHeiner

module.exports = function(grunt) {

    grunt.initConfig({

        foo: {
            options: {
                bar: {
                    minify: true,
                    scripts: ['a', 'b', 'c']
                }

NickHeiner

Is this worth it?

I've been editing this in response to comments below, but I will preserve the full revision history.

Referencing the function execution context via this is fairly common in JavaScript. I suspect some people like it because they come from a language like Java, and it’s familiar. However, I would argue that it in some cases introduces brittleness into one's code without bringing much benefit.

For example, let’s consider a contrived logger module:

function CountingLogger() {
 if (!(this instanceof CountingLogger)) return new CountingLogger;

NickHeiner

High level style in javascript.

Opinions are like assholes, every one has got one.

This one is mine.

Punctuation: who cares?

Punctuation is a bikeshed. Put your semicolons, whitespace, and commas where you like them.

NickHeiner

{
    "endpoint": "http://bower.opower.it/",
    "searchpath": [
        "https://bower.herokuapp.com"
    ],
    "registry": {
        "search": [
            "http://bower.opower.it/",
            "https://bower.herokuapp.com"
        ],

NickHeiner

A
└── B
│   └── E
└── C
│   └── E’
└── D
└── E’’

NickHeiner

"lodash": "^2.4.1"

NickHeiner

'use strict';

var sdk = require('internal-sdk');

module.exports = function(grunt) {
  sdk(grunt, {
    'project-type': 'widget'
  });
};

NickHeiner

grunt.registerTask(‘task-a’, function() {
  var result = doStuff();
  grunt.task.run(‘task-b’).then(function() {
    doMoreStuff();
  });
});

NickHeiner

grunt.registerTask(‘task-a’, function() {
  var result = doStuff();
  grunt.registerTask(‘after-task-b’, function() {
    doMoreStuff(result);
  });
  grunt.task.run(‘task-b’);
  grunt.task.run(‘after-task-b’);
});