Code review comment for ~fourdollars/pc-enablement/+git/oem-scripts:master

Revision history for this message
Yuan-Chen Cheng (ycheng-twn) wrote :

I am not a fan of re-use variable like this

+ if args.json:
+ args.json = json.load(args.json)

other look good. Up to the author to change it or not.

review: Approve

« Back to merge proposal