Enable Typedoc excludeNotExported

[benmccann]

This turns on excludeNotExported in typedoc

It also contains a lot of workarounds for TypeStrong/typedoc#393 to make it work

[benmccann]

Sigh... This works locally, so I'm not sure why the tests are failing on the VMs

[kurkle]

Probably need the exclude that was removed few days ago

[kurkle]

Try if that one helps: #7099 (comment)

[kurkle]

Ok, no other ideas to try.

[kurkle]

Oh, I read somewhere that you need to put the export on separate line in some cases (ie, end of file), when exporting a class as default (and using properties)

[benmccann]

Well that would defeat the entire purpose of this PR :-p

[benmccann]

This isn't worth spending more time on to me. I'd rather just not use excludeNotExported

[kurkle]

I was supposed to comment here about moving commonjs() lower in the plugins. But it was rollup-plugin-istanbul that is failing here. I think its using babel 6 still.

I think I got it working in #7125 (need to extract the chores from that)

[benmccann]

That worked! Thank you so much!!

[etimberg]

It is quite particular about those exports :|

[benmccann]

Yeah, hopefully they fix that bug in Typedoc, but I think it's generally better this way anyway

[kurkle]

Not sure about exporting the singleton classes. Maybe we should at least add a comment there that its only exported for typedocs?

[benmccann]

Ok, I just added some comments, but am just as happy to remove the export if you think that's preferrable

For the Animator, I think that class will mostly just be used internally and probably isn't something users would interact with, so removing the export and removing from the docs would be fine

For the Defaults, there's only one method: set. I just noticed that #7074 removed the _ prefix, so I just went ahead and removed the @private. I don't think the docs show this being used at all though and use alternate ways of setting the default. If you want to consider this class as internal for now as well I'm also happy to remove the export