Skip to content

feat(app): Add Gulp option #1001

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 297 commits into from
Closed

feat(app): Add Gulp option #1001

wants to merge 297 commits into from

Conversation

Awk34
Copy link
Member

@Awk34 Awk34 commented Jun 17, 2015

Needs:

  • Grunt parity
    • serve (needs testing)
    • build (needs testing)
    • watch (needs testing)
    • test
  • tests

DaftMonk and others added 30 commits September 21, 2014 18:30
Changes:
- add prompt for Jasmine or Mocha
- if Mocha choosen, prompt for Expect or Should
- use `<%= does() %>` to dynamically insert assertions (expect|should)
- add mocha variants for protractor tests
- add mocha options to protractor.conf
- remove `test/fixtures/(bower|package).json` from repo
- move runE2E functionality to runTest and simplify switch
- comment generator test functions
- use node `0.11.13` in travis due to issues with `0.11.14`+

Note:
Server-side jasmine test are needed to fully unify testing frameworks.
Once Jasmine tests are included for server, mocha dep can be removed
fully when selecting Jasmine.
docs(gen:endpoint): standardized comments
Conflicts:
	app/templates/Gruntfile.js
	app/templates/_bower.json
	test/fixtures/package.json
Conflicts:
	app/templates/server/config/express.js
@Awk34
Copy link
Member Author

Awk34 commented Aug 19, 2015

Hah, just noticed this PR is targeting master. I'll make a new one.

@Awk34 Awk34 closed this Aug 19, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.