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

Bump Traceur to version 0.0.90 #85

Closed

Conversation

baileyparker
Copy link

This bumps Traceur to the most recent version (right now 0.0.90).

All tests pass (I had to modify a few, but I ensured that none of that changes removed tests for API features that es6ify used).

I know this has been discussed in #79 and a fix was started in #80. I'm not well versed in es6ify or traceur internals, but the last discussion here was about 0.0.82, so perhaps more API changes have happened that now make es6ify's version bump more seamless.

To confirm that es6ify still works (beyond the safety of the tests), I went into a few of my projects that use es6ify and dropped in this new version. Everything compiled (and the projects still ran in browser).

Of course, I welcome any feedback, if there is an es6ify behavior that I've broken with this PR. I'll try my best to sort it out.

Thanks for maintaining this project! It's really useful to browserify community!

names: [],
mappings: sourceMap.mappings,
sourceRoot: sourceRootConverted,
sourcesContent: [
Copy link
Collaborator

Choose a reason for hiding this comment

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

So as far as I can tell this removes all our test coverage for source maps, which was the main sticking point in previous PRs :(. We can't break that...

@baileyparker baileyparker deleted the bump_tracuer_to_90 branch January 9, 2016 04:17
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.

2 participants