Skip to content
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.
This repository has been archived by the owner on Dec 7, 2018. It is now read-only.

Expose ping method on WebSocket driver. #187

Open
digitalextremist opened this issue Apr 8, 2015 · 2 comments
Open

Expose ping method on WebSocket driver. #187

digitalextremist opened this issue Apr 8, 2015 · 2 comments
Milestone

Comments

@digitalextremist
Copy link
Member

As requested in kenichi/angelo#52: add a forwardable delegator for ping to @driver.

@kenichi
Copy link
Member

kenichi commented Apr 8, 2015

or maybe just add :driver to the attr_accessor list?

@digitalextremist
Copy link
Member Author

@kenichi that's a possibility, I'd need to think about that. I'm leaning toward this being more of a duck type and abstraction layer, rather than encouraging people to directly access the driver and become dependent on driver-specific code some day.

Will ponder while also resolving #186, but currently over in Celluloid itself for a bit.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants