Skip to content
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.

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

Load input plugins at startup and call them directly when a command is received #191

Merged
merged 1 commit into from
Mar 15, 2016

Conversation

astorije
Copy link
Member

This extends #154 a bit more and is a first step into improving things (next step will add tests!) :-)

@xPaw, I know you have looked at this code recently, let me know if you can think of how I can make this better (the existing changes, as I'm leaving out other things that will need changing later on).

@astorije astorije added the Type: Feature Tickets that describe a desired feature or PRs that add them to the project. label Mar 14, 2016
@xPaw
Copy link
Member

xPaw commented Mar 14, 2016

Seems fine to me 👍

@maxpoulin64
Copy link
Member

Seems fine to me as well. 👍 and merging.

maxpoulin64 added a commit that referenced this pull request Mar 15, 2016
Load input plugins at startup and call them directly when a command is received
@maxpoulin64 maxpoulin64 merged commit c74811c into master Mar 15, 2016
@astorije astorije deleted the astorije/direct-inputs branch March 15, 2016 03:51
@astorije astorije added this to the 1.4.0 milestone Oct 8, 2016
@xPaw xPaw removed their assignment Mar 12, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: Feature Tickets that describe a desired feature or PRs that add them to the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants