Anaminus / rbxmk

A tool for processing Roblox files.
MIT License
109 stars 11 forks source link

Refactor AttrConfig as descriptor extension #68

Open Anaminus opened 2 years ago

Anaminus commented 2 years ago

Currently, there are several locations where a descriptor can be passed in, but an AttrConfig cannot. One option would be to combine RootDesc and AttrConfig under a common "config" type. However, AttrConfig only contains simple configuration data, which is very suitable for what extensions are designed for.

So, instead of maintaining a separate type, AttrConfig will be removed, and reimplemented as a descriptor extension:

{
    "$rbxmkFields": {
        "Attributes": {
            "Property": "AttributesSerialize"
        }
    }
}