Marc-André Cournoyer

We ignore `UnusuedSnippet` check as soon as a dynamic render is used: `{% render block %}` https://github.com/Shopify/theme-check/blob/master/lib/theme_check/checks/unused_snippet.rb#L20 Maybe this is not needed for `render`? Need to revisit this.

Check localized keys in {% schema %}

1

Should run all the translation checks on https://shopify.dev/docs/themes/sections#locales

Investigate bringing back UndefinedObjects for snippets

9

It was disabled by default in #160 because of issues with optional parameters. Possible there's a better way to make this all work out. But I can't think of any...

Implement https://github.com/Shopify/liquid-style-guide as checks

4

At quick glance, there are few things in there we could turn into new checks.

It would be awesome if offenses found in JSON pointed to the exact line number. Current situation: ``` sections/main-collection.liquid:98: suggestion: MatchingSchemaTranslations: settings.center_align_text.label missing translations for cs, da, de, es, fi,...

See list of deprecated properties on https://shopify.dev/docs/themes/liquid/reference/objects/deprecated-objects

Translation keys can have Liquid code in them to receive arguments: ``` "page": "Page {{ number }}", ``` We should run a few applicable checks on those (`SpaceInsideBraces`, `SyntaxError`, `UnknownFilter`).

This is only about translations validate. See #70 for validating the format of the file. Doesn't seem to be documented anywhere, but `settings_schema.json` accepts translations: ```json [ { "name": {...

See https://shopify.dev/docs/themes/settings#file-format-overview for format

Validate all the fields in {% schema %}

1

See https://shopify.dev/docs/themes/sections#using-section-schema-tags for the accepted structure. However, keep in mind that this page doesn't mention that all values that end up in the UI can be localized. Translation checks in...