Code review comment for lp://qastaging/~johannes.erdfelt/nova/bug747394

Revision history for this message
Rick Harris (rconradharris) wrote :

> 19 + for key in entries.keys():

Very minor, but it's more idiomatic to iterate directly over the `dict`:

    for key in entries:

Otherwise lgtm. Since the suggestion is so minor, marking as Approved anyway.

review: Approve

« Back to merge proposal