Closed evanweible-wf closed 10 years ago
With the camelCase refactor (https://github.com/Workiva/wGulp/pull/73), a reference to the old test:generate task was left in the watch:test task definition, causing all watch:test runs to fail.
test:generate
watch:test
Update references of test:generate to preTest.
preTest
jspm-testing
npm install && jspm install && gulp watch:test
gulp watch:test
npm install --save-dev ../../../wGulp
@maxwellpeterson-wf @trentgrover-wf @charliekump-wf
+1
+10
@charliekump-wf ready for merge + minor release
QA Resource Approval: +1
Merging into master.
Problem
With the camelCase refactor (https://github.com/Workiva/wGulp/pull/73), a reference to the old
test:generate
task was left in thewatch:test
task definition, causing allwatch:test
runs to fail.Solution
Update references of
test:generate
topreTest
.How to Test / +10
jspm-testing
examplenpm install && jspm install && gulp watch:test
(this installs the wGulp#master)gulp watch:test
failsnpm install --save-dev ../../../wGulp
(installs the local version of wGulp - the watchTestFix branch)gulp watch:test
again, should succeed@maxwellpeterson-wf @trentgrover-wf @charliekump-wf