ReactBinding Rewrite

[gaymeowing]

Rewrites ReactBinding to be cleaner, and also use less memory as its now properly using metatables.

• Improve error messages
• Removes use of pairs()
• Removes impl table jank
• Removes createSignal as its no-longer used
• Remove BindingInternal type as its useless (will be replacing with proper types for each prototype table and each type of binding in React Next, I just cant be bothered to do proper types when its going to be thrown out soon anyways)
• Rename BindingInternalApi table to ReactBinding
• Mark __subscribeToBinding as depricated as every binding now has a :_subscribe() method that should be used instead
• Makes RobloxComponentProps use _:subscribe()
• Make the update function not iterate twice when firing off callbacks
• Signal implementation for bindings now uses less memory, as it only uses one table for storing callbacks and nolonger creates connection objects

Checklist before submitting:

• Added/updated relevant tests
• Added/updated documentation

[gaymeowing]

Ran on:

Benchmark place: ReactBindingRewriteBenchmark.zip