Should automatically lint *.mjs files

[feross]

ES6 module support is coming to Node.js. That means a new file extension (*.mjs) that denotes an ES6 module file. standard should automatically lint this file extension, since that's what users will expect.

[yoshuawuyts]

Sounds reasonable. Side note tho: from talking to TSC members, this may change in the future. My understanding is that the `.mjs` extension was brought in place so the module spec could be implemented and tested, but now that that's happened, they'll be digging into ergonomics, which means figuring out compat with just a `.js` extension. It's unsure what will come from it, but yeah it might be prone to change. Kinda just sharing what I've overheard in recent weeks (:

…

On Tue, Oct 31, 2017 at 5:39 AM Feross Aboukhadijeh < ***@***.***> wrote: ES6 module support is coming to Node.js. That means a new file extension (*.mjs) that denotes an ES6 module file. standard should automatically lint this file extension, since that's what users expect. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#1009>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACWlehhJsMq-oQLwoPffDzX6epxVFN9Oks5sxqSZgaJpZM4QMMtq> .

[ematipico]

Yeah they've made a big of a mess

[feross]

@yoshuawuyts Cool, thanks for sharing. I've just started playing around with --experimental-modules. We'll keep an eye on this!

Edit: This blog post has the most useful info I've been able to find about ESM in Node.js: http://2ality.com/2017/09/native-esm-node.html

[stale]

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs.

[feross]

Only one repo started failing from this change (merge2) and it was because they had lint errors in their .mjs files and weren't manually running standard on them. So, this is a great change!

This will ship in standard 14.

[feross]

I just remembered that .cjs files are also coming to Node, so I'll add those by default too.