Add LocalizationsValidator using validation in Localized mixin

[danielpgross]

TL;DR

Migrated LocalizationsValidator module from old tool version, reimplementing most of it for much better performance.

On my machine, the new implementation takes ~150ms compared to ~22s for the old version. The speedup is thanks to using the already loaded localizations rather than loading them all again.

What changed?

• Added a validation method to the Localized mixin
• Added a new LocalizationsValidator module to validate translations across categories, attributes, and values
• Integrated validation into the generate_dist command

The old implementation checked for 2 cases:

• Localization key missing
• Localization key present, but name key missing for it

These are effectively the same check, since the name key being present requires the localization key itself to be present. In the new implementation, they are combined into a single check.

[danielpgross]

• Migrate generate_release command to new tool #543
• Add LocalizationsValidator using validation in Localized mixin #542 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[danielpgross]

Merge activity

• Jan 29, 5:47 PM EST: A user merged this pull request with Graphite.