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

Support nested import statements #48

Open
workflow opened this issue Sep 1, 2016 · 2 comments
Open

Support nested import statements #48

workflow opened this issue Sep 1, 2016 · 2 comments

Comments

@workflow
Copy link

workflow commented Sep 1, 2016

Hey there and thanks for this package!

Here's the problem: Doc builds are currently failing on any files that sport nested @import declarations, a new feature added in recent meteor versions. Example error:

ERROR: Unable to parse /opt/web/imports/api/companyInvestors/methods.js: Line 266: Illegal import declaration"

An upgrade to Meteor 1.3.5.1 (or higher) will fix this.

@emilianosantucci
Copy link

@workflow try with my pull request #49 ;)

@workflow
Copy link
Author

@emilianosantucci Thx for the pull request!

Contrary to what I would have expected, the problem persists with recent meteor versions. Could be a problem with jsdoc itself, or maybe ejs, since using nested @import declarations isn't valid ES6 (but valid Meteor).

@workflow workflow changed the title Updating Meteor version to support nested import statements Support nested import statements Dec 30, 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

No branches or pull requests

2 participants