g910-gkey-macro-support icon indicating copy to clipboard operation
g910-gkey-macro-support copied to clipboard

Full new install/uninstall/README files...

Open braoult opened this issue 4 years ago • 1 comments

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.

braoult avatar Jun 19 '20 11:06 braoult

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?

suabo avatar Oct 20 '22 16:10 suabo