Open Bug 1609061 Opened 6 years ago Updated 3 years ago

[Automated review] For `mozlint-l10n-conflicts` reporter add a link to the string in the l10n repo in the message

Categories

(Developer Infrastructure :: Source Code Analysis, enhancement, P4)

Product:
Developer Infrastructure
Component:
Source Code Analysis
Type:
enhancement

Tracking

(Not tracked)

People

(Reporter: jdescottes, Unassigned)

Details

Reviewbot reported 3 warnings on the following diff: https://phabricator.services.mozilla.com/D59837#1823875

Warning: Changes to string require a new ID: table.headers.cookies.expires [l10n: source-test-mozlint-l10n-conflicts]

But I think all the strings mentioned here are new?

It turns out the strings were removed from central in https://bugzilla.mozilla.org/show_bug.cgi?id=1554841, but they are still in the l10n repositories. So reviewbot is right, but it was hard to understand. Maybe we could add a link to the string in the l10n repo in the warning?

The priority flag is not set for this bug.
:Sylvestre, could you have a look please?

For more information, please visit auto_nag documentation.

Flags: needinfo?(sledru)
Type: defect → enhancement
Priority: -- → P4
Flags: needinfo?(sledru)
Summary: [Automated review] Reviewbot reports l10n warnings for new strings → [Automated review] For `mozlint-l10n-conflicts` reporter add a link to the string in the l10n repo in the message
Product: Firefox Build System → Developer Infrastructure
Severity: normal → S3
You need to log in before you can comment on or make changes to this bug.