Skip to content
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

Pass console object in to broccoli-babel-transpiler. #86

Merged
merged 1 commit into from
Aug 11, 2016

Conversation

rwjblue
Copy link
Member

@rwjblue rwjblue commented Aug 11, 2016

This will be used by broccoli-babel-transpiler to log warnings/deprecations to the console when a given plugin does not provide the correct caching information.

This will be used by babel/broccoli-babel-transpiler#89.

@stefanpenner
Copy link
Member

sg, in future versions of CLI we may want to have ui.toConsole() or something

@rwjblue
Copy link
Member Author

rwjblue commented Aug 11, 2016

Ya, that seems like a good idea.

@stefanpenner
Copy link
Member

for the future.

@rwjblue
Copy link
Member Author

rwjblue commented Aug 11, 2016

Build is failing because currently there is no way to distinguish options intended to be passed into babel itself, versus those intended for broccoli-babel-transpiler and when babel gets the console object passed here it throws an error.

Ideally, we should refactor the babel options to be passed in via options.babel or something, so that the other non-babel options (like console, includePolyfill, etc) can be in the root of options.

@rwjblue
Copy link
Member Author

rwjblue commented Aug 11, 2016

I guess I have to release broccoli-babel-transpiler first, then this...

@rwjblue
Copy link
Member Author

rwjblue commented Aug 11, 2016

Released [email protected] and rebased this to update to that version.

This will be used by broccoli-babel-transpiler to log
warnings/deprecations to the console when a given plugin does not
provide the correct caching information.
@rwjblue
Copy link
Member Author

rwjblue commented Aug 11, 2016

@stefanpenner / @Turbo87 - This is ready to go, but I don't have merge + release access here. If one of y'all could release (or add me) I would appreciate it...

@stefanpenner stefanpenner merged commit 0fdada6 into emberjs:master Aug 11, 2016
@stefanpenner
Copy link
Member

stefanpenner commented Aug 11, 2016

releasing as v5.1.8

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.

3 participants