yoshuawuyts / barracks

:mountain_railway: action dispatcher for unidirectional data flows
MIT License
177 stars 22 forks source link

Remove extra start() from docs #36

Closed timwis closed 8 years ago

timwis commented 8 years ago

I'm assuming this is a typo?

coveralls commented 8 years ago

Coverage Status

Coverage remained the same at 94.488% when pulling 7c242f0ab5dac9612ca0788aab1fcc766004cb22 on timwis:docs-remove-extra-start into bc485ffef1bef0b242534e7afbe99a3fb1afa43b on yoshuawuyts:master.

yoshuawuyts commented 8 years ago

Hah, perhaps confusingly, it's not! - So what's going on here is that:

const createSend = store.start({ noSubscriptions: true })

fires up everything but the subscriptions, and then

store.start()

fires up everything that hasn't been started yet; thus firing up the subscriptions. I agree that we should probably document this better; can imagine this does look confusing haha.