JSHint will fail the build on errors

Moved currently failing files to individual excludes so they can be fixed individually.
wet-boew · Aug 21, 2012 · 28db4e2 · 28db4e2
1 parent 7119520
commit 28db4e2
Show file tree

Hide file tree

Showing 2 changed files with 31 additions and 4 deletions.
diff --git a/build/build-tasks.xml b/build/build-tasks.xml
@@ -91,7 +91,7 @@
 	</target>
 
 	<target name="-jshint">
-		<jshint dir="${src.dir}" fail="false" globalsFile="${jshint.globals.file}">
+		<jshint dir="${src.dir}" fail="true" globalsFile="${jshint.globals.file}">
 			<report type="xml" destfile="${build.dir}/jshint.out.xml" />
 			<include name="**/*.js"/>
 			<exclude name="**/*.min.js"/>

diff --git a/src/js/build.xml b/src/js/build.xml
@@ -145,12 +145,39 @@
 	<target name="test" depends="-jshint"/>
 
 	<target name="-jshint">
-		<jshint dir="${src.dir}" fail="false" globalsFile="${jshint.globals.file}" options="strict:false">
+		<jshint dir="${src.dir}" fail="true" globalsFile="${jshint.globals.file}" options="strict:false">
 			<report type="xml" destfile="${build.dir}/jshint.out.xml" />
 			<include name="**/*.js"/>
-			<include name="dependencies/calendar.js"/>
 			<exclude name="**/*.min.js"/>
-			<exclude name="dependencies/**/*.js"/>
+			<exclude name="i18n/formvalid/methods_nl.js"/>
+			<exclude name="jquery.mobile/jquery.mobile.js"/>
+			<exclude name="polyfills/datalist.js"/>
+			<exclude name="polyfills/detailssummary.js"/>
+			<exclude name="polyfills/html5shiv.js"/>
+			<exclude name="polyfills/localstorage.js"/>
+			<exclude name="polyfills/slider.js"/>
+			<exclude name="workers/menubar.js"/>
+			<exclude name="workers/parser.table.js"/>
+			<exclude name="workers/webwidget.js"/>
+			<exclude name="dependencies/bookmark.js"/>
+			<exclude name="dependencies/browsertweaks.js"/>
+			<exclude name="dependencies/colorbox.js"/>
+			<exclude name="dependencies/easytabs.js"/>
+			<exclude name="dependencies/equalheights.js"/>
+			<exclude name="dependencies/hashchange.js"/>
+			<exclude name="dependencies/hoverintent.js"/>
+			<exclude name="dependencies/json.js"/>
+			<exclude name="dependencies/matchMedia.js"/>
+			<exclude name="dependencies/metadata.js"/>
+			<exclude name="dependencies/outside.js"/>
+			<exclude name="dependencies/pie.js"/>
+			<exclude name="dependencies/prettify.js"/>
+			<exclude name="dependencies/prettify/**/*.js"/>
+			<exclude name="dependencies/raphael.js"/>
+			<exclude name="dependencies/resize.js"/>
+			<exclude name="dependencies/validate.js"/>
+			<exclude name="dependencies/validateAdditional.js"/>
+			<exclude name="dependencies/xregexp.js"/>
 		</jshint>
 	</target>
 </project>