New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make Communicator and SensorManager objects singletons #59

Open

TuckerHaydon opened this issue Apr 27, 2017 · 1 comment

Assignees

Collaborator

TuckerHaydon commented Apr 27, 2017

No need for multiple objects. Make a singleton so that it is not required to be injected into all commands.

TuckerHaydon self-assigned this

TuckerHaydon changed the title ~~Make communicator object a singleton~~ Make Communicator and SensorManager objects singletons

Collaborator Author

TuckerHaydon commented Apr 28, 2017

They should be singletons but still need to inject them for testing

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment