Closed kellyredding closed 11 years ago
@jcredding this is really just for the lastest commit Once we merge #106, I'll rebase and the diff will be nice and clean. Heads up.
@jcredding ok this has been rebased and can be reviewed cleanly.
@kellyredding - Looks good. :boom:
You now configure a Runner instance. There are a couple reasons for this.
One, the user may need to do custom Runner initialization and its easier to let them do it and config Assert with an instance. It's never good to be creating instances of things on behalf of the users.
Two, this is more consistant with how you configure a View or a Suite - you just pass an instance.
@jcredding ready for review