Closed Bug 1537994 Opened 9 months ago Closed 8 months ago

[wpt-sync] Sync PR 15980 - Always use json (and never ujson) to serialize manifest

Categories

(Testing :: web-platform-tests, enhancement, P4)

enhancement

Tracking

(firefox68 fixed)

RESOLVED FIXED
mozilla68
Tracking Status
firefox68 --- fixed

People

(Reporter: wptsync, Unassigned)

References

()

Details

(Whiteboard: [wptsync downstream])

Sync web-platform-tests PR 15980 into mozilla-central (this bug is closed when the sync is complete).

PR: https://github.com/web-platform-tests/wpt/pull/15980
Details from upstream follow.

Sam Sneddon <me@gsnedders.com> wrote:

Always use json (and never ujson) to serialize manifest

ujson.dump is scarcely quicker than json.dump, but produces different output which is annoying for those who commit the generated manifest.

The PR was not expected to affect any tests, but the try push wasn't a success. Check the try results for infrastructure issues
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1176223b2164
[wpt PR 15980] - Always use json (and never ujson) to serialize manifest, a=testonly
Pushed by james@hoppipolla.co.uk:
https://hg.mozilla.org/integration/mozilla-inbound/rev/0f4726afc673
[wpt PR 15980] - Always use json (and never ujson) to serialize manifest, a=testonly
Status: NEW → RESOLVED
Closed: 8 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.