Closed DanielSSilva closed 6 years ago
@tylerl0706 can you please check if the change is now correct? I'm kind of confused with github now... I've made the change on my patch-1
branch. I thought it would notify it here
Turns out CLA is not required for this PR. So we can safely merge this in. Thanks for contributing! 😃
As I've talked with @Tyler, it should be clear that since this requires hardware interaction, the module must be installed with high privileges. I've also indicated the path to the module as an "hint". I assumed that it will always be the same since it's installed as root (correct me if I'm wrong).