maryrosecook/coquette — pull request #24 by eric-brechemier (2013-11-14 10:20:31+00:00)
Title: Asteroid in Advanced Demo uses undocumented module runner

Since the Runner is not described in the README and is only used by the Entities module to plan entities creation and destruction at this point, it feels like coquette.runner is not part of the public API.

In this pull request, I propose to remove the reference to the runner from the demo. I saw no visible impact when calling the destroy() method of the asteroid immediately (which delays the destruction to the next tick anyway) instead of waiting one more tick before calling destroy() (and planning the actual destruction one tick later).

The alternative would be to make the runner explicitly part of the API by describing it in the README.

More generally, it is not really clear to me which parts of the publicly accessible properties and methods are fair game, and which ones should not be used because they could evolve or be removed in the future. I will open a separate issue about this concern.