1497503 - Add `locales` to [[paths]] in project config and compare-locales

Status: RESOLVED FIXED

(Localization Infrastructure and Tools :: compare-locales, enhancement)

Description

[Axel Hecht [:Pike]]

We should add support for `locales` in `[[paths]]`, to enable a particular directory only for a subset of locales.

The specified list of locales is assumed to be a subset.

Comment 1

[Axel Hecht [:Pike]]

Attachment: documentation

Turns out compare-locales can already do that. I was confused by the docs stating otherwise, so fixing the docs.

Code in c-l:

https://phabricator.services.mozilla.com/source/compare-locales/browse/default/compare_locales/paths/configparser.py$81-82
https://phabricator.services.mozilla.com/source/compare-locales/browse/default/compare_locales/paths/project.py$63-64
https://phabricator.services.mozilla.com/source/compare-locales/browse/default/compare_locales/paths/files.py$31-35
https://phabricator.services.mozilla.com/source/compare-locales/browse/default/compare_locales/tests/paths/test_files.py$120-163

Comment 2

[Francesco Lodolo [:flod]]

Comment on attachment 9015530 [details] [diff] [review]
documentation

Review of attachment 9015530 [details] [diff] [review]:
-----------------------------------------------------------------

::: docs/fileformat.rst
@@ +98,5 @@
>  For bilingual file formats, it's acceptable to just specify the ``l10n`` path,
>  though it's encouraged to define the ``reference`` to enable more tests.
>  
> +To limit a particular path pattern to a subset of locales, you can specify
> +an option ``locales`` entry:

option -> optional? The latter seems more natural

Comment 3

[Axel Hecht [:Pike]]

Documentation fixed. Also, local setup fixed, which I hadn't done yet after we created the /l10n/ upstream repo, including adjusting the config of rtd to actually point to the official location.