amzn / style-dictionary

A build system for creating cross-platform styles.
https://styledictionary.com
Apache License 2.0
3.87k stars 543 forks source link

fix(formats): added nested property to json/asset #1000

Open jduthon opened 1 year ago

jduthon commented 1 year ago

Issue #, if available:

Description of changes:

We are getting a lot of warnings during tokens' generation mentioning name collisions.

I started looking into this and discovered the nested format idea, which works great for the formats where it's set up (and we had to add it to a few of our formats), but then I realized I was still getting errors during asset files generation with the json/format format. Looking at the code + at its output, it seems like this format is a nested format too, but I am guessing it was missed during the time where nested formats where introduced.

So this PR is just making the json/format officially a nested format, so there won't be anymore wrong warnings for it.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.