g910-gkey-macro-support
g910-gkey-macro-support copied to clipboard
Full new install/uninstall/README files...
Looks ok now, for me. Easy to use. One command to install, one to uninstall. However, I still did not remove the (useless for me) create_config script, which should be done. I will just remove it in my fork, without pull req. Unfortunately, this could become a permanent fork.
I like the idea of having a uninstall script. Maybe you can fix the quality of your commit by removing not needed changes to readme (whitespaces and line breaks) so the check don't fails. Also I think there is too much changing at once. I would remove the selection of keyboard layout on install because it will need to get updated with every new layout like it is the case right now. Maybe make another pull request for that and try to read the current system layout to use in install on default?