Merge pull request #102 from Financial-Times/handle-bower-branches

allow branches of bower components

lib/verify-layout-deps.js

@@ -39,9 +39,19 @@ module.exports = function(opts) {
 				}
 
 				// test to see if the component is a compatible version
-				var appSemver = require(path.join(process.cwd(), 'bower_components', dep, '.bower.json')).version;
-				if (!semver.satisfies(appSemver, layoutBowerDeps[dep])) {
-					reject('This app needs to install a version of ' + dep + ' compatible with the semver ' + layoutBowerDeps[dep] + ' in order to render layouts');
+				var bowerJson = require(path.join(process.cwd(), 'bower_components', dep, '.bower.json'));
+				var appSemver = bowerJson.version;
+				if (!appSemver) {
+					if (bowerJson._resolution.type === 'branch') {
+						console.warn('This app is using a non-versioned release of ' + dep);
+						console.warn('It\'s ok to do so while experimenting, but longer term try to revert to using a semvered version');
+					} else {
+						reject('Unknown bower resolution for ' + dep + '. Use a branch or semver');
+					}
+				} else {
+					if (!semver.satisfies(appSemver, layoutBowerDeps[dep])) {
+						reject('This app needs to install a version of ' + dep + ' compatible with the semver ' + layoutBowerDeps[dep] + ' in order to render layouts');
+					}
 				}
 			}
 		});