Skip to content

fix(gulp): add babel runtime options #1472

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

Merged
merged 2 commits into from
Dec 8, 2015

Conversation

david-mohr
Copy link
Contributor

without es7.classProperties, babel won't compile the default controller template

@Awk34
Copy link
Member

Awk34 commented Dec 8, 2015

I use the gulp-load-plugins module, which finds all npm modules we use that start with gulp- and adds them on the plugins object that I have at the top of the file; so you don't have to change plugins.babel to babel.

@@ -392,7 +395,7 @@ gulp.task('build:client', ['transpile:client', 'styles', 'html'], () => {

let assets = plugins.useref.assets({searchPath: ['client', '.tmp']});

return gulp.src(paths.mainView)<% if(filters.jade) { %>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

split this change out to a separate commit, since it's unrelated to the other changes

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Would you like a new PR, or just a separate commit? (sorry, pretty new to this)

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just pull it into a separate commit. If you know how to rebase, I'd suggest it. If not, just make a new PR.

@Awk34 Awk34 added the bug label Dec 8, 2015
without `es7.classProperties`, babel won't compile the default controller template
update `build:client` so that `gulp.src` points to `paths.client.mainView`
Awk34 added a commit that referenced this pull request Dec 8, 2015
fix(gulp): add babel runtime options, fix a path
@Awk34 Awk34 merged commit 237e3c2 into angular-fullstack:canary Dec 8, 2015
@Awk34
Copy link
Member

Awk34 commented Dec 8, 2015

Thanks!

@david-mohr david-mohr deleted the canary branch December 8, 2015 23:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants