868178 - l10n comments in templates should not be case sensitive

Status: VERIFIED FIXED

(www.mozilla.org :: Bedrock, defect, P3)

Description

[Paul [:pmac] McLanahan]

To provide some context for a string you can add a comment in a template like:

{# L10n: This is for flying #}

It has to start with exactly "L10n" and is case sensitive. It should accept any combination of case for the L and N.

Comment 1

[Dan Poirier [:dpoirier]]

Status update:

* Have gotten this feature merged into tower: https://github.com/clouserw/tower/pull/20

* Have PR open to update playdoh-lib to use the new version of tower: https://github.com/mozilla/playdoh-lib/pull/42

* Final step will be to update bedrock to use the new version of playdoh-lib.

Comment 2

[Paul [:pmac] McLanahan]

Nice job fixing it the "Right Way" :)

Comment 3

[[github robot]]

Commits pushed to master at https://github.com/mozilla/bedrock

https://github.com/mozilla/bedrock/commit/2720952a6e54fe4f7d6c012c6c67f8fe69b23f6f
Bug 868178 - Make L10N translator comment tag case-insensitive

* Update playdoh-lib to version that has that change in tower
* Fix one tower import due to a method having been moved

https://github.com/mozilla/bedrock/commit/041eb0ae13ba504779a956263a7d9dac364644d8
Merge pull request #1131 from dpoirier/bug-868178-case-insensitive-l10n-comments

Bug 868178 - Make L10N translator comment tag case-insensitive