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

Add support for export ... from syntax #8

Closed
wants to merge 1 commit into from

Conversation

olalonde
Copy link
Contributor

No description provided.

@mrjoelkemp
Copy link
Collaborator

@olalonde Thanks so much for contributing! Awesome work and great catch.

I just published a patch release. Feel free to bring that to Madge.

@olalonde
Copy link
Contributor Author

Thanks 😄

@olalonde
Copy link
Contributor Author

I'm trying to use this in madge but for some reason it doesn't seem to work in my test....

pahen/madge@master...olalonde:reexport

If you have any idea what I'm doing wrong, would be helpful :)

@olalonde
Copy link
Contributor Author

@mrjoelkemp
Copy link
Collaborator

My first hunch was that you didn't do npm i --save detective-es6@1.1.5 and the shinkwrap didn't get updated. Nice find on the import sniff within madge proper.

mrjoelkemp added a commit to dependents/node-dependents-editor-backend that referenced this pull request Jun 25, 2016
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