@loganfsmyth What do you think about this vs ExportNamespaceDeclaration? (adding to reviewers as you had some concerns)

For what it's worth, I think that ExportAllDeclaration is proper because it takes all members from "source" then exports them. The word "all" affects to source rather than destination, so we can write other export declarations even if ExportAllDeclaration existed.

If ExportAllDeclaration means "it exports an ambiguous number of elements (in other words, it needs to load more files to decide the number of exported elements)," then using ExportNamedDeclaration is reasonable. But ModuleSpecifier has local property always, we should fix it.

Any update about this?

Just fyi this PR is holding up support in projects like acorn.

One more friendly ping here.

Friendly ping here, as this is holding up support for TS3.8 features in typescript-eslint (and thus in prettier as well).