Bug 1789329 Comment 5 Edit History

Note: The actual edited comment in the bug view page will always show the original commenter’s name and original timestamp.

Original comment by
on 
Unfortunately, the dev run revealed that there's other module related stuff that we don't support, which isn't totally surprising, although I had thought we had made the JS analyzer not care so much about JS problems, but I guess it does!  The specific error that broke the indexing run:
```
/home/ubuntu/mozsearch/scripts/js-analyze.js:326:12 uncaught exception: Unexpected statement: ExportDeclaration
```

I reverted the fix.  We'll want to add a representative module to the "tests" corpus with a check.
Revision 1 by
on 
Unfortunately, the dev run revealed that there's other module related stuff that we don't support, which isn't totally surprising, although I had thought we had made the JS analyzer not care so much about JS problems, but I guess it does!  The specific error that broke the indexing run (on `browser/actors/AboutPluginsParent.sys.mjs):
```
/home/ubuntu/mozsearch/scripts/js-analyze.js:326:12 uncaught exception: Unexpected statement: ExportDeclaration
```

I reverted the fix.  We'll want to add a representative module to the "tests" corpus with a check.

Back to Bug 1789329 Comment 5