red-blox / zap

A lightning fast networking solution for roblox.
https://zap.redblox.dev
MIT License
95 stars 15 forks source link

Freeze module returns #97

Closed witchiestwitchery closed 4 months ago

witchiestwitchery commented 4 months ago

Makes the return table of the generated modules frozen, including the nested tables for events

witchiestwitchery commented 4 months ago

Small nit with regards to code style, you use table.freeze({ sometimes and sometimes you use table.freeze {. Can you convert all of them to be table.freeze({?

Only did table.freeze({ for the studio return, as table.freeze {} :: T in the new solver currently produces a type error. Will do it if you still think its necessary, I just think having parentheses would unnecessarily clutter up how the return looks.