This is totally solid, though you could set up more tasks that do more granular things; e.g.:

```js
// Default task
grunt.registerTask('default', ['test', 'clean', 'concat', 'uglify']);

// Testing
grunt.registerTask('test', ['jshint', 'qunit']);
```

No particular need for it, but if you're thinking this Gruntfile might grow, this would be a good way to break things down some more.