[TS2503] Should we override Animated to be a TS namespace?

[alloy]

Currently in DT Animated is defined as a namespace, whereas it's a plain object in the Flow types.

import { Animated } from "react-native"
const value: Animated.Value = new Animated.Value(42)

[alloy]

I’m on the fence as to what to do here.

[alloy]

I recall that @TheSavior previously had mentioned that RN core is open to accepting patches to change code to use ES6 syntax, but that they simply hadn’t had a strong need to do so yet. Perhaps the Animated APIs would be a good case to change upstream so that we don’t need to do any real conversions, but that means that for the time being we should perform that change in the TS typings with an override.

[TheSavior]

What would you propose the change to the upstream code looks like?

[alloy]

I decided to fix it in TS instead, seeing as namespaces are a TS only thing. d385982