Update in-tree version of compare-locales to 1.1

RESOLVED FIXED

Status

RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: Pike, Unassigned)

Tracking

Details

Attachments

(1 attachment)

(Reporter)

Description

2 years ago
I found a bug in compare-locales which didn't remove broken content from l10n files during merge.

Let's update to compare-locales 1.1.

Also 1.1 has better test coverage, which made me find that bug.
(Reporter)

Comment 1

2 years ago
Created attachment 8778250 [details]
bug 1292557, update compare-locales to 1.1,

Notable changes:

Bug 1238150 - Don't consider trailing comments as junk
Bug 1292215 - Drop Junk entities from merged content, more tests

Review commit: https://reviewboard.mozilla.org/r/69564/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/69564/
Attachment #8778250 - Flags: review?(francesco.lodolo)
Attachment #8778250 - Flags: review?(francesco.lodolo) → review+
Comment on attachment 8778250 [details]
bug 1292557, update compare-locales to 1.1,

https://reviewboard.mozilla.org/r/69564/#review66718

Can't spot anything wrong with the code changes.

Comment 3

2 years ago
Pushed by axel@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/e648eb9685a0
update compare-locales to 1.1, r=flod

Comment 4

2 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/e648eb9685a0
Status: NEW → RESOLVED
Last Resolved: 2 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.