How to export a module variable in CoffeeScript

[fknipp]

Maybe there might be better places in the guide to put this information, but it should be written somewhere.

[stubailo]

@fknipp can you add an entry to the changelog? https://github.com/meteor/guide#changelog

[fknipp]

You're welcome :)

[stubailo]

Thanks!

[GeoffreyBooth]

I just stumbled across this. This section is just wrong. The syntax is not different in CoffeeScript; CoffeeScript supports the import and export keywords, nearly identically to ES2015. What this PR is referring to is CommonJS module.exports, though it’s confusing CommonJS’ global exports object with the export keyword in CoffeeScript/ES2015.

This was written up here: https://docs.meteor.com/packages/modules.html#CoffeeScript. Clearly that’s hard to find, so perhaps we can replace this section with a link to that one?

[abernix]

@GeoffreyBooth That sounds fine to me. Can you do a PR or shall I?

[lorensr]

Thanks Geoffrey, fixed in abedc3b

[GeoffreyBooth]

@lorensr Thanks for tackling this, I haven’t had time just yet. In that commit it looks like the exports.Lists = ListsCollection 'lists' example is still there, after the link you added. I would think that this should become export Lists = new ListsCollection 'lists' or similar, i.e. using the export keyword (and ES2015 modules) rather than exports (and CommonJS). Even for CoffeeScript I think we should be encouraging the import and export keywords, and ES2015 modules, over CommonJS. Backticks aren’t required anymore.

[lorensr]

Oops thanks, fixed!