`commits.commits` is really puzzling for an uninformed reader. An attempt to see what the first `commits` is fails: it's a closure parameter with no type annotation. Further digging in Dart Editor reveals that it's a `CommitPushEntry`. All this makes for unproductive reading/reviewing. Could you add a type and rename it to `pushEntry`? In general, type annotations are very helpful.