1276254 - Replace unnecessary usage of read() when loading json from files

Status: RESOLVED FIXED

(Tree Management :: Treeherder, defect, P3)

Description

[Ed Morley [:emorley]]

Another minor cleanup that's been sitting in my stashes for a while (having a spring clean).

This pattern:

  with open(...) as f:
      foo = json.loads(f.read())

Can be replaced with:

  with open(...) as f:
      foo = json.load(f)

Comment 1

[GitHub Autolander Bot]

Attachment: [treeherder] mozilla:rm-json-loads-read > mozilla:master

Comment 2

[Treeherder GitHub Bugbot]

Commit pushed to master at https://github.com/mozilla/treeherder

https://github.com/mozilla/treeherder/commit/b6677fd5c699a676203baf1e38905510cc93cfa1
Bug 1276254 - Remove unnecessary usage of .read() when json decoding

By using `json.load()` instead of `json.loads()` we can pass the file
object directly, rather than having to `.read()` it first.