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

feat: Initial command line #129

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from
Draft

feat: Initial command line #129

wants to merge 3 commits into from

Conversation

ids1024
Copy link
Member

@ids1024 ids1024 commented Mar 13, 2022

This seems to be working now. I don't know if printing info messages by default is ideal in the CLI, or of this is the best way to handle selection with multiple boards.

I guess translations need to be updated, since this moves some strings to the backend so the importing logic can be there.

Leaving as a draft.

ids1024 added 3 commits March 6, 2022 13:05
Implements #84.

Uses clap derive for argument parsing.

Could be improved, since resetting layout (for instance) unnecessarily
involves the backend fetching the layout for all connected keyboard.

Printing info messages by default seems good with the GUI, a bit more
out of place with CLI commands.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant