labofsahil/github-event-dataset-2012 huggingface dataset:
would be nice to give the example an id and then use a xref here. I think we use this approach quite consistently throughout the docs.
for consistency with imapchew, please add an "|| null" here.  null may still break things downstream, but it reduces the edge cases a little bit if we have to deal with this.  (I have seen ridiculous cases relating to this in gloda bugs with both 0 from's and 2 from's...)
the sleep is to avoid a busy loop
on one of the systems i'm testing on, this sleep is necessary to avoid eating 100% cpu (falls to almost 0% with this 1ms sleep)

on other systems, this part of the code is rarely triggered, and even if it is, it's only a 1ms sleep between events, which is really low (see a 1 sec sleep below) and won't impact command latency
This description is pretty small, can you expand upon what the helper actually does? i.e. automatically scaling images, involving cache if necessary.. that kind of thing
Fixed in next PR.
I'm not sure view should depend on what controllers that call to render the view. Because what I think Rails follow convention to has a partial of an object in collection view itself, and that's why in Rails introduces a shortcut to render collection by just call `render @articles`. @jordanbyron, what do you think?
I don't think the renaming of this plays well with deserializing old builds.
insert whitespace before loop
Lets rename this since its not really a constant
add test for non null activity id
Is the iVar still needed? I thought `- (BOOL)isFirstViewControllerViewOverdraggedToLeft {...}` now always contains a valid result.
I think there is a CS problem in this diff 

Please run `php check_cs fix`
IMHO, Not worth it for a one off.