rojo-rbx / rojo

Rojo enables Roblox developers to use professional-grade software engineering tools
https://rojo.space
Mozilla Public License 2.0
908 stars 172 forks source link

Don't serialize `emitLegacyScripts` if it's `None` #835

Closed Dekkonot closed 5 months ago

Dekkonot commented 5 months ago

Right now, if someone runs rojo fmt-project on a project that doesn't have emitLegacyScripts specified, it will emit a file with the default value (true right now). This is undesirable because it forces users into specifying a redundant field if they want to format their project.

This PR fixes that by simply removing the default for the field from Serde. Since we specify a default when we read it anyway, this has no impact on actual usage but makes fmt-project behave the same between 7.3 and 7.4.