Closed Bug 1537994 Opened 7 years ago Closed 7 years ago

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

Categories

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

Product:
Testing
Component:
web-platform-tests
Type:
enhancement

Tracking

(firefox68 fixed)

Status:
RESOLVED FIXED
Milestone:
mozilla68
Tracking Flags:
Tracking Status
firefox68 --- fixed

People

(Reporter: wpt-sync, Unassigned)

References

(
URL
)

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

https://hg.mozilla.org/mozilla-central/rev/1176223b2164
https://hg.mozilla.org/mozilla-central/rev/0f4726afc673

Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox68: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla68
You need to log in before you can comment on or make changes to this bug.