netaneld122 / firmware-variables

Python library for controlling UEFI variables in Windows.

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Remove destructive code from Readme

IlayRosenberg opened this issue · comments

It's not okay to just put code that destroys your main boot entry in the readme file with no warnings or documentation / description.
Don't you think it'll be better to include in the readme only code that doesn't modify the system state? All other code examples can be moved to the examples directory.
Also I think all code examples (both in the readme file and in the examples directory) should be documented

I planned to improve the documentation so that will be my next patch,
but generally, since 50% of the library API is modifying the system state by design, I don't see any problem in showing examples that modify the system state.
I will definitely add a warning and perhaps make the examples less destructive.