Closed Bug 1597258 Opened 10 months ago Closed 10 months ago

Make l10n-lint report to phabricator with warnings on

Categories

(Firefox Build System :: Task Configuration, task)

task
Not set
normal

Tracking

(firefox72 fixed)

RESOLVED FIXED
mozilla72
Tracking Status
firefox72 --- fixed

People

(Reporter: Pike, Assigned: Pike)

References

()

Details

Attachments

(2 files)

When we added moz-l10n-lint originally, we didn't make it create the artifact that reviewbot uses to annotate phab.

Let's change that.

Let's also try to split the behavior we want on codereview and on regular automation, by adding a separate job for each. That should also help the current lack of pre- and post-linting aka https://github.com/mozilla/code-review/issues/32.

Attachment #9109633 - Attachment description: bug 1597258, create special l1nt for try, and trigger that, r=Bastien → Bug 1597258, create special l1nt for try, and trigger that, r=Bastien
Pushed by axel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/0c1efa982f7b
create special l1nt for try, and trigger that, r=bastien
Assignee: nobody → l10n
Status: NEW → RESOLVED
Closed: 10 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla72
Pushed by axel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/b4b10ae558b9
follow-up to disable always-target for try-only l10n-conflicts job, r=tomprince
You need to log in before you can comment on or make changes to this bug.