Skip to content

Conversation

@Swiip
Copy link
Owner

@Swiip Swiip commented Apr 20, 2015

This changes has started as an answer of #481. An impact after another, several things has been done.

  • Change the configuration model, there is now a gulp\conf.js which contains the conf. It ables to read conf values from outsides of gulp (especially karma) and remove complexity of exporting a function in all gulp files.
  • Remove TypeScript sortOutput.json and simply concat files in the build
  • Add karma-angular-filesort to make karma able to sort angular files
  • Remove gulp/unit-tests.js list files complexity, no more needs of gulp to sort files for karma
  • Add list files computation in karma.conf.js in order to make it works outside of gulp (karma start should work out of the box now), fix WebStorm karma config problems #481, fix although unit test is setup the test suite is empty... #473
  • Remove path computation with + for path.join, fix Use path.join instead of concatenating paths. #487
  • Added some comments in gulpfile.js and gulp/conf.js

zckrs pushed a commit that referenced this pull request Apr 22, 2015
@zckrs zckrs merged commit b45dd84 into master Apr 22, 2015
@Swiip Swiip deleted the karma-config branch September 9, 2015 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use path.join instead of concatenating paths. WebStorm karma config problems although unit test is setup the test suite is empty...

4 participants