Problem with default export after update to 2.1.0

[adascal]

Hi, after updating from version 2.0.1 to 2.1.0 I start getting error for code like:

export default from './Map.jsx';

Stack trace:

Node must be provided when reporting error if location is not provided
AssertionError: Node must be provided when reporting error if location is not provided
at EventEmitter.module.exports.api.report (/node_modules/eslint/lib/eslint.js:970:20)
at RuleContext.report (/node_modules/eslint/lib/rule-context.js:144:21)
at EventEmitter.Program:exit (/node_modules/eslint-plugin-import/lib/rules/prefer-default-export.js:69:19)
at emitOne (events.js:101:20)
at EventEmitter.emit (events.js:188:7)
at NodeEventGenerator.leaveNode (/node_modules/eslint/lib/util/node-event-generator.js:49:22)
at CodePathAnalyzer.leaveNode (/node_modules/eslint/lib/code-path-analysis/code-path-analyzer.js:628:23)
at CommentEventGenerator.leaveNode (/node_modules/eslint/lib/util/comment-event-generator.js:110:23)
at Controller.leave (/node_modules/eslint/lib/eslint.js:930:36)
at Controller.__execute (/node_modules/estraverse/estraverse.js:397:31)

[yvele]

I was going to open the exact same issue.

eslint@3.9.1 with eslint-plugin-import@2.0.1

[adascal]

eslint@3.9.1 with eslint-plugin-import@2.0.1 work fine for me...
Check installed module, maybe it's 2.1.0? :)

[yvele]

@adascal I'm using the exact same versions as you and I'm having the same error with an index.jshaving export foo from "./foo";

[ljharb]

@adascal you're using proposed syntax https://github.com/leebyron/ecmascript-export-default-from - which isn't part of the language, and is only stage 1. Are you using babel-eslint for this?

@yvele export notDefault from 'path' is in the language. export default from 'path' is not, which is what the OP is using.

[yvele]

@ljharb Yep. I'm not using default but I still have the Node must be provided.. error

PS: eslint@3.9.1 with eslint-plugin-import@2.0.1

[ljharb]

Ah, gotcha, thanks for clarifying.

[adascal]

@ljharb, yes, I'm using babel-eslint

[benmosher]

Hmm, I wonder if it's a babel-eslint change... not sure what would get into that situation. Regardless, we've got a bug here. Thanks for reporting!

[r3nya]

So … We can use this way:

export { default } from './Map.jsx';

instead

export default from './Map.jsx';

cc @adascal