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

Fix the callback function being called twice #5

Closed
wants to merge 1 commit into from

Conversation

koistya
Copy link

@koistya koistya commented Jul 13, 2016

Why the System.import handler is being executed after .catch(callback) Is this by design? module is undefined at that stage and leads to unhandled exception.

@codecov-io
Copy link

codecov-io commented Jul 13, 2016

Current coverage is 89.38%

No coverage report found for master at 3146059.

Powered by Codecov. Last updated by 3146059...59261b5

@jescalan
Copy link

Simple fix here, the catch code is just incorrect promise handling. PR looks good to me. I also ran into an issue with the error messages that come out of this in webpack2.

Copy link
Member

@sokra sokra left a comment

Choose a reason for hiding this comment

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

Looks good

@SpaceK33z
Copy link
Member

SpaceK33z commented Dec 13, 2016

@koistya, could you fix the merge conflict? Tried to do it myself via GitHub's new fancy web editor for merge conflicts, but it's buggy and won't let me :(.

@appden
Copy link

appden commented Mar 23, 2017

I opened PR #15 which includes this patch by @koistya with the merge issue resolved.

@sokra
Copy link
Member

sokra commented Apr 24, 2017

see #15

@sokra sokra closed this Apr 24, 2017
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.

None yet

6 participants