-
Notifications
You must be signed in to change notification settings - Fork 1.2k
feat(gulp): add Gulp support #1190
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
Conversation
}[val]; | ||
} | ||
}, { | ||
type: "list", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What do you think about moving this to projectPrompts
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
that would probably make more sense XD
@@ -82,7 +82,7 @@ var AngularFullstackGenerator = yeoman.generators.Base.extend({ | |||
this.log('# Client\n'); | |||
|
|||
this.prompt([{ | |||
type: 'list', | |||
type: "list", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should be single quotes here.
76750d1
to
d2325ce
Compare
0ce70d9
to
c5c00b9
Compare
f2dff11
to
8c0deba
Compare
to enable, run `yo angular-fullstack` with the `--gulp` flag
I was testing out your new gulp file and here are the changes I made to get
And in case you need them, here are my build selections.
Thanks for all the great work! |
@david-mohr if you think there's a problem, feel free to submit a PR |
So, GULP is implemented? |
I'm currently working on another PR based on my testing and porting some more Grunt tasks. |
@david-mohr awesome, I look forward to it |
Needs: